Update to latest versions of Bootsrap/JQuery/JQueryUI and Datetimepicker#124
Update to latest versions of Bootsrap/JQuery/JQueryUI and Datetimepicker#124herrtunante wants to merge 3 commits intomasterfrom
Conversation
| var PRODUCTION = function () { | ||
| try { | ||
| return "production" === 'production'; | ||
| } catch (e) { |
There was a problem hiding this comment.
no-unreachable: Unreachable code.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
There was a problem hiding this comment.
no-unreachable: Unreachable code.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| }); | ||
| Object.defineProperty(val, 'node', { | ||
| get: function get() { | ||
| if (!IS_DOM) return; |
There was a problem hiding this comment.
getter-return: Expected to return a value in method 'get'.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
There was a problem hiding this comment.
no-control-regex: Unexpected control character(s) in regular expression: \x1f, \x1f.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
There was a problem hiding this comment.
no-extra-semi: Unnecessary semicolon.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| function _typeof(obj) { | ||
| "@babel/helpers - typeof"; | ||
|
|
||
| return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function (obj) { |
There was a problem hiding this comment.
💬 2 similar findings have been found in this PR
no-func-assign: '_typeof' is a function.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/fontawesome.js | 46 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/fontawesome.js | 152 |
Visit the Lift Web Console to find more details in your report.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| if (typeof document !== 'undefined') _DOCUMENT = document; | ||
| if (typeof MutationObserver !== 'undefined') _MUTATION_OBSERVER = MutationObserver; | ||
| if (typeof performance !== 'undefined') _PERFORMANCE = performance; | ||
| } catch (e) {} |
There was a problem hiding this comment.
💬 8 similar findings have been found in this PR
no-empty: Empty block statement.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/solid.js | 15 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery-ui.js | 10518 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 753 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 3399 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 4506 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 6411 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 8982 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 10080 |
Visit the Lift Web Console to find more details in your report.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| window.removeEventListener('resize', instance.update, passive); | ||
| } | ||
| }; | ||
| } // eslint-disable-next-line import/no-unused-modules |
There was a problem hiding this comment.
💬 10 similar findings have been found in this PR
import/no-unused-modules: Definition for rule 'import/no-unused-modules' was not found.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1162 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1335 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1417 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1470 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1673 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1819 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1908 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1969 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1983 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/datepicker/popper.js | 1988 |
Visit the Lift Web Console to find more details in your report.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| } // Check if a regular expression validates the attribute. | ||
|
|
||
|
|
||
| return allowedAttributeList.filter(attributeRegex => attributeRegex instanceof RegExp).some(regex => regex.test(attributeName)); |
There was a problem hiding this comment.
💬 19 similar findings have been found in this PR
opt.semgrep.regex_dos: Ensure that the regex used to compare with user supplied input is safe from regular expression denial of service.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/bootstrap/bootstrap.bundle.js | 5301 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/bootstrap/bootstrap.bundle.js | 5301 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/bootstrap/bootstrap.js | 3482 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/bootstrap/bootstrap.js | 3475 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/bootstrap/bootstrap.js | 3475 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery-ui.js | 2761 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery-ui.js | 18039 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 836 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 853 |
| collect-server/src/main/resources/org/openforis/collect/designer/templates/collectearth/earthFiles/jquery/jquery.js | 853 |
Showing 10 of 19 findings. Visit the Lift Web Console to see all.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| var PRODUCTION = function () { | ||
| try { | ||
| return "production" === 'production'; | ||
| } catch (e) { |
There was a problem hiding this comment.
no-unreachable: Unreachable code.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
There was a problem hiding this comment.
no-unreachable: Unreachable code.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| }); | ||
| Object.defineProperty(val, 'node', { | ||
| get: function get() { | ||
| if (!IS_DOM) return; |
There was a problem hiding this comment.
getter-return: Expected to return a value in method 'get'.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| function _typeof(obj) { | ||
| "@babel/helpers - typeof"; | ||
|
|
||
| return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function (obj) { |
There was a problem hiding this comment.
💬 2 similar findings have been found in this PR
no-func-assign: '_typeof' is a function.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-webapp/src/main/webapp/earthFiles/datepicker/fontawesome.js | 46 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/fontawesome.js | 152 |
Visit the Lift Web Console to find more details in your report.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| if (typeof document !== 'undefined') _DOCUMENT = document; | ||
| if (typeof MutationObserver !== 'undefined') _MUTATION_OBSERVER = MutationObserver; | ||
| if (typeof performance !== 'undefined') _PERFORMANCE = performance; | ||
| } catch (e) {} |
There was a problem hiding this comment.
💬 8 similar findings have been found in this PR
no-empty: Empty block statement.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-webapp/src/main/webapp/earthFiles/datepicker/solid.js | 15 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery-ui.js | 10518 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 753 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 3399 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 4506 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 6411 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 8982 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 10080 |
Visit the Lift Web Console to find more details in your report.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| window.removeEventListener('resize', instance.update, passive); | ||
| } | ||
| }; | ||
| } // eslint-disable-next-line import/no-unused-modules |
There was a problem hiding this comment.
💬 10 similar findings have been found in this PR
import/no-unused-modules: Definition for rule 'import/no-unused-modules' was not found.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1162 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1335 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1417 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1470 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1673 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1819 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1908 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1969 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1983 |
| collect-webapp/src/main/webapp/earthFiles/datepicker/popper.js | 1988 |
Visit the Lift Web Console to find more details in your report.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| } // Check if a regular expression validates the attribute. | ||
|
|
||
|
|
||
| return allowedAttributeList.filter(attributeRegex => attributeRegex instanceof RegExp).some(regex => regex.test(attributeName)); |
There was a problem hiding this comment.
💬 19 similar findings have been found in this PR
opt.semgrep.regex_dos: Ensure that the regex used to compare with user supplied input is safe from regular expression denial of service.
🔎 Expand here to view all instances of this finding
| File Path | Line Number |
|---|---|
| collect-webapp/src/main/webapp/earthFiles/bootstrap/bootstrap.bundle.js | 5301 |
| collect-webapp/src/main/webapp/earthFiles/bootstrap/bootstrap.bundle.js | 5301 |
| collect-webapp/src/main/webapp/earthFiles/bootstrap/bootstrap.js | 3482 |
| collect-webapp/src/main/webapp/earthFiles/bootstrap/bootstrap.js | 3475 |
| collect-webapp/src/main/webapp/earthFiles/bootstrap/bootstrap.js | 3475 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery-ui.js | 2761 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery-ui.js | 18039 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 836 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 853 |
| collect-webapp/src/main/webapp/earthFiles/jquery/jquery.js | 853 |
Showing 10 of 19 findings. Visit the Lift Web Console to see all.
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
|
SonarCloud Quality Gate failed. |








No description provided.