Formatting in get_order_parcel_string().
authorIain Patterson <me@iain.cx>
Mon, 13 May 2013 08:49:20 +0000 (04:49 -0400)
committerIain Patterson <me@iain.cx>
Mon, 13 May 2013 12:48:07 +0000 (08:48 -0400)
Make sure the output of get_order_parcel_string() doesn't include any
leading or trailing colons.

lib/functions.php

index fe442f8..ecaf2ad 100644 (file)
   function get_order_parcel_string($order) {
     global $parcel_sizes, $parcel_contents;
 
-    $parcel_size = "";
+    $parcel_size = null;
     for ($i = 0 ; $i < count($parcel_sizes); $i++) {
       if ($order->getParcel() & (1 << $i)) {
         $parcel_size = $parcel_sizes[$i];
       if ($order->getParcel() & (1 << $i)) $selected[] = $parcel_contents[$i];
     }
 
-    return implode(": ", array($parcel_size, implode(", ", $selected)));
+    $ret = implode(": ", array($parcel_size, implode(", ", $selected)));
+    $ret = preg_replace('/^: /', '', $ret);
+    $ret = preg_replace('/: $/', '', $ret);
+
+    return $ret;
   }
 
   function get_order_displayname($order) {