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. |