dynamic / foxystripe

code/pages/ProductHolder.php

Doc Comments +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
      * Products function.
      * 
      * @access public
-     * @return array
+     * @return PaginatedList
      */
     public function ProductList($limit = 10)
     {

Spacing +4 added lines, -4 removed lines

@@ -129,14 +129,14 @@ 
     {
         $config = SiteConfig::current_site_config();
 
-        if ($config->ProductLimit>0) {
+        if ($config->ProductLimit > 0) {
             $limit = $config->ProductLimit;
         }
 
         if ($config->MultiGroup) {
             $entries = $this->Products()->sort('SortOrder');
-        } else {
-            $filter = '"ParentID" = ' . $this->ID;
+        }else {
+            $filter = '"ParentID" = '.$this->ID;
 
             // Build a list of all IDs for ProductGroups that are children
             $holderIDs = $this->ProductGroupIDs();
@@ -147,7 +147,7 @@ 
                 if ($filter) {
                     $filter .= ' OR ';
                 }
-                $filter .= '"ParentID" IN (' . implode(',', $holderIDs) . ")";
+                $filter .= '"ParentID" IN ('.implode(',', $holderIDs).")";
             }
 
             $order = '"SiteTree"."Title" ASC';