Skip to content

Reports - fix replacing text with multiple file paths, hotfix bootstrap menu with many tools#239

Merged
zdenekdrahos merged 2 commits into
masterfrom
reports-20210525
May 25, 2021
Merged

Reports - fix replacing text with multiple file paths, hotfix bootstrap menu with many tools#239
zdenekdrahos merged 2 commits into
masterfrom
reports-20210525

Conversation

@zdenekdrahos

Copy link
Copy Markdown
Member
  • fix replacing text with multiple file paths
  • hotfix displaying both phpmetrics reports in menu for gh-pages

Before: Unexpected token: ;, line: 3, col: 7, file: src/invalid.php. #0 /long-path/vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/AbstractPHPParser.php...
After: Unexpected token: ;, line: 3, col: 7, file: src/invalid.php. #0 vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/AbstractPHPParser.php...

replaceAll is not supported in older browsers, but it should be fine
- https://caniuse.com/?search=replaceAll
- https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String/replaceAll
Good enough at the moment. It needs different approach when user has many tools and smaller screen.
@zdenekdrahos zdenekdrahos merged commit b7aa6ca into master May 25, 2021
@zdenekdrahos zdenekdrahos deleted the reports-20210525 branch May 25, 2021 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant