View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005741 | Composr | core | public | 2024-04-28 18:57 | 2024-07-22 21:09 |
| Reporter | Patrick Schmalstig | Assigned To | |||
| Severity | Feature-request | ||||
| Status | non-assigned | Resolution | open | ||
| Product Version | |||||
| Fixed in Version | |||||
| Summary | 0005741: Consider is_readable instead of file_exists where applicable | ||||
| Description | is_readable may be a better option performance-wise than file_exists if we are checking for a file and also planning to read it. We may need to make our own cms_is_readable like we have cms_is_writable. | ||||
| Tags | Roadmap: Over the horizon, Type: Performance | ||||
| Time estimation (hours) | |||||
| Sponsorship open | |||||
|
|
This is true, but it's also a micro-optimization. Should only be done in critical path code. Trying to force it everywhere would lead to counter-intuitive code (the naming is a bit less clear), and yet another coding standard to enforce (we probably already have too many). |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-04-28 18:57 | Patrick Schmalstig | New Issue | |
| 2024-04-28 19:17 | Patrick Schmalstig | Tag Attached: Roadmap: Over the horizon | |
| 2024-07-22 21:08 | Chris Graham | Tag Attached: Type: Performance | |
| 2024-07-22 21:09 | Chris Graham | Note Added: 0008874 |
