View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0005379 | OXID eShop (all versions) | 4.07. Source code, Test | public | 2013-08-30 09:49 | 2023-11-21 09:19 |
Reporter | arvydas_vapsva | Assigned To | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 4.7.6 / 5.0.6 | ||||
Summary | 0005379: oxArticleList and select * from does not employ lazy loading | ||||
Description | In oxArticleList there are places in source code, where all fields are selected from oxarticles table, e.g. "select * from.." or "select $sArticleTable.* from...". This should be optimized to e.g.: "select ".$this->getBaseObject()->getSelectFields()." from ..." | ||||
Tags | No tags attached. | ||||
Theme | Not defined | ||||
Browser | All | ||||
PHP Version | Not defined | ||||
Database Version | Not defined | ||||
|
Alternatively lazy loading in objects inside an oxList could be activated. Currently there seems to be a problem with this. So if you don't know which fields you will need, you use selectString('select * from ...'). That's probably not the intended usage. |
|
This is now the default way how it is done since many years. I only found 2 places in the core, where * is used and those were very specific cases, where it makes sense. |