CRM-18701 tidy up, do not pass variables around as a reference when they are not...

diff --git a/CRM/Price/BAO/LineItem.php b/CRM/Price/BAO/LineItem.php
index 26c64950422ea396cc612cefaa534ec1f758520b..f13a13cee704d7963eed1aabf08869863d454fc7 100644 (file)
--- a/CRM/Price/BAO/LineItem.php
+++ b/CRM/Price/BAO/LineItem.php
@@ -323,22 +323,18 @@ AND li.entity_id = {$entityId}
    * @param array $params
    *   Reference to form values.
    * @param array $fields
-   *   Reference to array of fields belonging.
+   *   Array of fields belonging.
    *                          to the price set used for particular event
    * @param array $values
    *   Reference to the values array(.
    *   this is
    *                          lineItem array)
-   *
-   * @return void
    */
-  public static function format($fid, &$params, &$fields, &$values) {
+  public static function format($fid, $params, $fields, &$values) {
     if (empty($params["price_{$fid}"])) {
       return;
     }
 
-    $optionIDs = implode(',', array_keys($params["price_{$fid}"]));
-
     //lets first check in fun parameter,
     //since user might modified w/ hooks.
     $options = array();