Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 3.0.0, 3.4.3
-
Fix Version/s: 3.5.0
-
Component/s: None
-
Labels:None
Description
Module::changeServicePackage describes that it may return an array of updated service fields. However, when this method is called in Services::edit, no return value is used. This causes the service to not be updated to its new service fields when changing the package.
Additionally, a subsequent call to Module::addService or Module::editService is made, which does return service fields to be set to the service. These service fields would override any returned from Module::changeServicePackage if they were used.
Question:
Should the service fields from Module::changeServicePackage be set on the service and included in the subsequent call to Module::addService or Module::editService calls?