adobe / aem-core-wcm-components

Standardized components to build websites with AEM.
https://docs.adobe.com/content/help/en/experience-manager-core-components/using/introduction.html
Apache License 2.0
730 stars 736 forks source link

[SITES-16535] Links with hashes "#" in button components are URL enco… #2598

Closed cristic83 closed 10 months ago

cristic83 commented 10 months ago

…ded when saved

Q                       A
Fixed Issues? Fixes SITES-16535
Patch: Bug Fix?
Minor: New Feature?
Major: Breaking Change?
Tests Added + Pass? Yes
Documentation Provided Yes (code comments and or markdown)
Any Dependency Changes?
License Apache License, Version 2.0
sonarcloud[bot] commented 10 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

codecov[bot] commented 10 months ago

Codecov Report

Merging #2598 (9555bc5) into main (b4327c3) will increase coverage by 0.00%. The diff coverage is 100.00%.

:exclamation: Current head 9555bc5 differs from pull request most recent head ba9de94. Consider uploading reports for the commit ba9de94 to get more accurate results

@@            Coverage Diff            @@
##               main    #2598   +/-   ##
=========================================
  Coverage     87.33%   87.34%           
  Complexity     2648     2648           
=========================================
  Files           232      232           
  Lines          7076     7078    +2     
  Branches       1073     1073           
=========================================
+ Hits           6180     6182    +2     
  Misses          359      359           
  Partials        537      537           
Files Coverage Δ
...cq/wcm/core/components/internal/link/LinkUtil.java 86.95% <100.00%> (+0.28%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more