da wo es sinnvoll ist, z.b sCustomers(), sGetOrders(), sGetOpenOrders(), sSuppliers(), sCategories(), sFullArticles(), sArticles() mal wird ein parameter erwartet, mal nicht. mal kann parameter ein assoziatives array sein, wo der schlüssel “where” ausgewertet wird, mal nicht. wie wärs, wenn all diese methoden folgende parameter erwarten: ($pkIDs = array(), $where = null, $offset = null, $limit = null) woraus dann sql resultiert: $sql = 'select … '; if ($where) $sql .= 'where ’ .$where; if ($offset != null && $limit != null) { $sql .= " limit $offset,$limit"; } oder sowas in der art.
hi, danke für die Anregung. Wir legen generell Wert auf sauberen und strukturiertem Code. Wir müssen natürlich auch an gewisse Abhängigkeiten denken, sodass weiterhin alte Plugins und Anpassungen funktionieren. Generell liegt uns das Thema Refaktorisierung am Herzen und wir sind bemüht mit jeder Version eine gute Code Basis zu veröffentlichen. Wir werden Ihren Hinweis mit in die Planung einbeziehen. Ob es allerdings 1:1 so umgesetzt wird, können wir jetzt natürlich noch nicht sagen. Grüße, Marcel