Bug: 14146 Remove redundant unserialize.

It's already done in the concrete class.
horde · Nov 8, 2015 · ed2486c · ed2486c
1 parent cf1f326
commit ed2486c
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/whups/lib/Driver.php b/whups/lib/Driver.php
@@ -315,13 +315,6 @@ public function getAllTicketAttributesWithNames($ticket_id)
 
         $attributes = array();
         foreach ($ta as $id => $attribute) {
-            try {
-                $value = Horde_Serialize::unserialize(
-                    $attribute['attribute_value'],
-                    Horde_Serialize::JSON);
-            } catch (Horde_Serialize_Exception $e) {
-                $value = $attribute['attribute_value'];
-            }
             $attributes[$attribute['attribute_id']] = array(
                 'id'         => $attribute['attribute_id'],
                 'human_name' => $attribute['attribute_name'],
@@ -330,8 +323,9 @@ public function getAllTicketAttributesWithNames($ticket_id)
                 'readonly'   => false,
                 'desc'       => $attribute['attribute_description'],
                 'params'     => $attribute['attribute_params'],
-                'value'      => $value);
+                'value'      => $attribute['attribute_value']);
         }
+
         return $attributes;
     }