This is an automated email from the ASF dual-hosted git repository.

shamrick pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git


The following commit(s) were added to refs/heads/master by this push:
     new 337ee579b8 DSR Capability Checkbox auto change (#7420)
337ee579b8 is described below

commit 337ee579b853c32e3ca732f3b76ee11c423d82fa
Author: Rima Shah <[email protected]>
AuthorDate: Wed Mar 29 15:47:22 2023 -0600

    DSR Capability Checkbox auto change (#7420)
    
    * Added isRequest for Regional Max Origin Connections
    
    * Fixed the current and previous value box for DS capabilities.
    
    * Addressed review comments.
---
 .../form/deliveryService/FormDeliveryServiceController.js  | 14 ++++++++++++++
 .../form/deliveryService/form.deliveryService.DNS.tpl.html |  8 +++++++-
 .../deliveryService/form.deliveryService.HTTP.tpl.html     |  8 +++++++-
 3 files changed, 28 insertions(+), 2 deletions(-)

diff --git 
a/traffic_portal/app/src/common/modules/form/deliveryService/FormDeliveryServiceController.js
 
b/traffic_portal/app/src/common/modules/form/deliveryService/FormDeliveryServiceController.js
index fd9c007685..be24049365 100644
--- 
a/traffic_portal/app/src/common/modules/form/deliveryService/FormDeliveryServiceController.js
+++ 
b/traffic_portal/app/src/common/modules/form/deliveryService/FormDeliveryServiceController.js
@@ -132,6 +132,20 @@ var FormDeliveryServiceController = 
function(deliveryService, dsCurrent, origin,
        };
        $scope.arrayCompare = arrayCompare;
 
+       /**
+        * This function is called when capability is updated on a DSR
+        */
+       function capabilityChange() {
+               const cap = [];
+               for (const [key, value] of 
Object.entries($scope.selectedCapabilities)) {
+                       if (value) {
+                               cap.push(key);
+                       }
+               }
+               deliveryService.requiredCapabilities = cap;
+       }
+       $scope.capabilityChange = capabilityChange;
+
        /**
         * This function is called on 'change' events for any and all TLS 
Version
         * inputs, and sets validity states of duplicates.
diff --git 
a/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.DNS.tpl.html
 
b/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.DNS.tpl.html
index 256c7e5bc7..76ce0c9d34 100644
--- 
a/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.DNS.tpl.html
+++ 
b/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.DNS.tpl.html
@@ -361,11 +361,12 @@ under the License.
                                         name="capability-{{$index}}"
                                         
ng-model="selectedCapabilities[capability.name]"
                                         
ng-checked="selectedCapabilities[capability.name]"
+                                        ng-change="capabilityChange()"
                                     />
                                     <a 
ng-href="/#!/server-capabilities/edit?name={{capability.name}}" 
target="_blank"> {{capability.name}}</a>
                                 </div>
                             </div>
-                            <aside class="current-value" 
ng-if="settings.isRequest" 
ng-show="{{arrayCompare(deliveryService.requiredCapabilities, 
dsCurrent.requiredCapabilities)}}">
+                            <aside class="current-value" 
ng-if="settings.isRequest" 
ng-show="arrayCompare(deliveryService.requiredCapabilities, 
dsCurrent.requiredCapabilities)">
                                 <h3 ng-if="open()">Current Value</h3>
                                 <h3 ng-if="!open()">Previous Value</h3>
                                 <pre>{{::dsCurrent.requiredCapabilities}}</pre>
@@ -536,6 +537,11 @@ under the License.
                                     ng-model="deliveryService.regional"
                                     style="max-width: max-content; min-width: 
1em;"
                             />
+                            <aside class="current-value" 
ng-if="settings.isRequest" ng-show="deliveryService.regional != 
dsCurrent.regional">
+                                <h3 ng-if="open()">Current Value</h3>
+                                <h3 ng-if="!open()">Previous Value</h3>
+                                <pre>{{::dsCurrent.regional}}</pre>
+                            </aside>
                         </div>
                     </div>
                     <div class="form-group" ng-class="{'has-error': 
hasError(cacheConfig.maxRequestHeaderBytes), 'has-feedback': 
hasError(cacheConfig.maxRequestHeaderBytes)}">
diff --git 
a/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.HTTP.tpl.html
 
b/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.HTTP.tpl.html
index 4ff551a41f..a1b06577e1 100644
--- 
a/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.HTTP.tpl.html
+++ 
b/traffic_portal/app/src/common/modules/form/deliveryService/form.deliveryService.HTTP.tpl.html
@@ -361,11 +361,12 @@ under the License.
                                         name="capability-{{$index}}"
                                         
ng-model="selectedCapabilities[capability.name]"
                                         
ng-checked="selectedCapabilities[capability.name]"
+                                        ng-change="capabilityChange()"
                                     />
                                     <a 
ng-href="/#!/server-capabilities/edit?name={{capability.name}}" 
target="_blank">{{capability.name}}</a>
                                 </div>
                             </div>
-                            <aside class="current-value" 
ng-if="settings.isRequest" 
ng-show="{{arrayCompare(deliveryService.requiredCapabilities, 
dsCurrent.requiredCapabilities)}}">
+                            <aside class="current-value" 
ng-if="settings.isRequest" 
ng-show="arrayCompare(deliveryService.requiredCapabilities, 
dsCurrent.requiredCapabilities)">
                                 <h3 ng-if="open()">Current Value</h3>
                                 <h3 ng-if="!open()">Previous Value</h3>
                                 <pre>{{::dsCurrent.requiredCapabilities}}</pre>
@@ -536,6 +537,11 @@ under the License.
                                     ng-model="deliveryService.regional"
                                     style="max-width: max-content; min-width: 
1em;"
                             />
+                            <aside class="current-value" 
ng-if="settings.isRequest" ng-show="deliveryService.regional != 
dsCurrent.regional">
+                                <h3 ng-if="open()">Current Value</h3>
+                                <h3 ng-if="!open()">Previous Value</h3>
+                                <pre>{{::dsCurrent.regional}}</pre>
+                            </aside>
                         </div>
                     </div>
                     <div class="form-group" ng-class="{'has-error': 
hasError(cacheConfig.maxRequestHeaderBytes), 'has-feedback': 
hasError(cacheConfig.maxRequestHeaderBytes)}">

Reply via email to