Details
-
Type:
Sub-task
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 4.5.0-b1
-
Component/s: None
-
Labels:None
Description
The SSL Store plugin should be updated to follow the new cron task design.
That is, these should be updated:
- CronTasks::getByKey. Pass in all arguments specifying this task is a plugin task
- CronTasks::add. Pass task_type and dir instead of plugin_dir
- CronTasks::delete. Replace this deprecated method with CronTasks::deleteTask
Additionally, the plugin should no longer perform order synchronization. The SSL Store module should be updated to do that instead. This plugin should remove that functionality and delete it's associated cron task.
NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron AND adding an email template. Instead, the plugin can be eliminated updated to only set the email template and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.
The CronTasks model has changed in CORE-2435, and some methods have been updated that the plugin calls.
These should be updated:
- CronTasks::getByKey. Pass in all arguments specifying this task is a plugin task
- CronTasks::add. Pass task_type and dir instead of plugin_dir
- CronTasks::delete. Replace this deprecated method with CronTasks::deleteTask
Activity
Tyson Phillips (Inactive)
created issue -
Tyson Phillips (Inactive)
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Epic Link |
|
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked higher |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.4.0 Sprint 1 [ 64 ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked lower |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.4.0 Sprint 1 [ 64 ] | 4.4.0 Sprint 2 [ 59 ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked higher |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.4.0 Sprint 3 [ 59 ] | 4.4.0 Sprint 3, 4.4.0 Sprint 4 [ 59, 71 ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked higher |
Tyson Phillips (Inactive)
made changes -
| Description |
The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
*NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron. Instead, the plugin can be eliminated and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.*
The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.4.0 Sprint 3, 4.4.0 Sprint 4 [ 59, 71 ] | 4.4.0 Sprint 3, 4.4.0 Sprint 4, 4.4.0 Sprint 5 [ 59, 71, 72 ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked higher |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.4.0 Sprint 3, 4.4.0 Sprint 4, 4.4.0 Sprint 5 [ 59, 71, 72 ] | 4.4.0 Sprint 3, 4.4.0 Sprint 4, 4.4.0 Sprint 5, 4.4.0 Sprint 6 [ 59, 71, 72, 73 ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked higher |
Tyson Phillips (Inactive)
made changes -
| Description |
*NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron. Instead, the plugin can be eliminated and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.*
The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
*NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron AND adding an email template. Instead, the plugin can be -eliminated- updated to only set the email template and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.*
The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
Tyson Phillips (Inactive)
made changes -
| Description |
*NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron AND adding an email template. Instead, the plugin can be -eliminated- updated to only set the email template and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.*
The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
The SSL Store plugin should be updated to follow the new cron task design.
That is, these should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ Additionally, the plugin should no longer perform order synchronization. The SSL Store module should be updated to do that instead. ---- *NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron AND adding an email template. Instead, the plugin can be -eliminated- updated to only set the email template and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.* The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
Tyson Phillips (Inactive)
made changes -
| Assignee | Jonathan Reissmueller [ jonathan ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked lower |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.4.0 Sprint 3, 4.4.0 Sprint 4, 4.4.0 Sprint 5, 4.4.0 Sprint 6 [ 59, 71, 72, 73 ] | 4.4.0 Sprint 3, 4.4.0 Sprint 4, 4.4.0 Sprint 5 [ 59, 71, 72 ] |
Tyson Phillips (Inactive)
made changes -
| Rank | Ranked lower |
Tyson Phillips (Inactive)
made changes -
| Summary | TheSSLStore: Update CronTasks model calls to support new design | The SSL Store plugin: Update CronTasks model calls to support new design |
Tyson Phillips (Inactive)
made changes -
| Description |
The SSL Store plugin should be updated to follow the new cron task design.
That is, these should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ Additionally, the plugin should no longer perform order synchronization. The SSL Store module should be updated to do that instead. ---- *NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron AND adding an email template. Instead, the plugin can be -eliminated- updated to only set the email template and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.* The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
The SSL Store plugin should be updated to follow the new cron task design.
That is, these should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ Additionally, the plugin should no longer perform order synchronization. The SSL Store module should be updated to do that instead. This plugin should remove that functionality and delete it's associated cron task. ---- *NOTE: The SSL Store plugin appears to only exist for the purpose of performing these automation tasks via cron AND adding an email template. Instead, the plugin can be -eliminated- updated to only set the email template and the module can be updated to support the cron actions now. In this case, the plugin may need to be updated to check whether the module supports cron tasks, and if so, do not perform it's cron tasks.* The CronTasks model has changed in These should be updated: * _CronTasks::getByKey_. Pass in all arguments specifying this task is a plugin task * _CronTasks::add_. Pass _task_type_ and _dir_ instead of _plugin_dir_ * _CronTasks::delete_. Replace this deprecated method with _CronTasks::deleteTask_ |
Tyson Phillips (Inactive)
made changes -
| Parent |
|
|
| Issue Type | Task [ 3 ] | Sub-task [ 5 ] |
Tyson Phillips (Inactive)
made changes -
| Epic Link |
|
Tyson Phillips (Inactive)
made changes -
| Fix Version/s | 4.5.0-b1 [ 11108 ] |
| Remaining Estimate | 0 minutes [ 0 ] | |
| Time Spent | 58 minutes [ 3480 ] | |
| Worklog Id | 11674 [ 11674 ] |
Tyson Phillips (Inactive)
made changes -
| Status | Open [ 1 ] | Closed [ 6 ] |
| Resolution | Fixed [ 1 ] |
