servicecatalog / oscm

An Enterprise-ready Cloud Services Management Software.
https://openservicecatalogmanager.org
Apache License 2.0
38 stars 14 forks source link

Task#1256 remove console warnings #1265

Closed farmakiG closed 3 years ago

farmakiG commented 3 years ago

Changes in this Pull Request:

Mandatory checks:

To be checked by the reviewer:

To be checked by the pull request owner:

Cross repository and core changes:

OSCM Build References:

Screenshot (if applicable): Screenshot Subscription Tabs Console warning: TabsWarning

Screenshot no warning: tabsNoWarning

Screenshot warning hexToHSL hexToHSL

Screenshot no warning: noWarning


This change is Reviewable

codecov[bot] commented 3 years ago

Codecov Report

Merging #1265 (4e5b288) into master (07eaaaf) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1265   +/-   ##
=========================================
  Coverage     55.47%   55.47%           
  Complexity    14427    14427           
=========================================
  Files          1375     1375           
  Lines         80473    80473           
  Branches       8598     8598           
=========================================
  Hits          44643    44643           
  Misses        33140    33140           
  Partials       2690     2690           
farmakiG commented 3 years ago

oscm-portal/WebContent/marketplace/account/subscriptionDetails.xhtml, line 74 at r1 (raw file):

Previously, GoebelL (Lorenz Goebel) wrote…
Are you sure restorePaymentTab is not needed? a) Did you test the payment tab in existing payed subscriptions ? b) I assume the method is still needed elsewhere as you didn't remove it completely. - If yes, did you check this part as well? - If no, please remove it completely.

You are right, this function is necessary, but only in the case that the #paymentTab is there.