View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005827 | OXID eShop (all versions) | 4. ------ eShop Core ------- | public | 2014-07-20 23:45 | 2014-07-24 10:04 |
| Reporter | leofonic | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Product Version | 4.8.6 / 5.1.6 | ||||
| Summary | 0005827: oxMaintenance is not executed by random visitor | ||||
| Description | In oxmaintenance.php, description is: * Maintenance task handler. Maintenance tasks are called periodically, by crontab * or by random visitor. But in oxshopcontrol, only updateUpcomingPrices is called directly: startProfile('executeMaintenanceTasks'); oxNew("oxarticlelist")->updateUpcomingPrices(); stopProfile('executeMaintenanceTasks'); | ||||
| Steps To Reproduce | Would be better if oxmaintenance::execute() would call oxmaintenance, and only if usecron is not set. This way oxmaintenance could be extended for own maintenance tasks. Schedule should not depend on type of call. Right now updateUpcomingPrices is forced if called by cron. It should be possible to call cron e.g. every 5 minutes and execute maintenance tasks if needed based on schedule. It would be very good if a possibility for own schedules would be implemented in standard. | ||||
| Tags | No tags attached. | ||||
| Theme | Azure | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
|
|
I mean: "would be better if *oxshopcontrol* would call oxmaintenance" |
|
|
This is a feature request. Waiting for PO decision. Also, we suggest: For crontab usage - create as many cron*.php files as you need for now to be able to execute calls on different times. If it is not necessary, just extend oxMaintenance::execute() method. For performance reasons we would not recommend to add more actions in shop control. It is better to use cron jobs. Update price function is more exception for system which cannot use cron jobs. Sorry for “or by random visitor” comment, we will improve it. |
|
|
OK i understand that it's not intended to add more tasks to random visitor, so from my point of view this bug can be closed. |
|
|
just changed comment |
