Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix/1130improve behaviour on ambiguous x path match #1137

Closed
wants to merge 12 commits into from
Closed

Fix/1130improve behaviour on ambiguous x path match #1137

wants to merge 12 commits into from

Conversation

leonrohne27
Copy link
Contributor

Fixes: #1130

Implements:

  • Now logs a warning instead of an error + stacktrace
  • user is not being asked anymore if he wants to continue despite the error
  • The code scans the child elements of the template document and uses seenElements to check whether an element has already been merged. If an element already exists, it is skipped.

@leonrohne27
devonfw#913: Added support for SquirrelSQL
9a57a74
@leonrohne27
devonfw#913
e4d0874
@leonrohne27
Merge branch 'main' into implement/913-add-support-for-squirelsql
258eb09
@leonrohne27
devonfw#913
e890ede
@leonrohne27
devonfw#1130: improve behaviour on ambigous XPath match
bfd4cb2
@leonrohne27
Delete cli/src/main/resources/nls/Help_de.properties
3cdad08
@leonrohne27
Delete cli/src/main/java/com/devonfw/tools/ide/url/updater/UpdateMana…
23ad65d 
…ger.java
@leonrohne27
Delete cli/src/main/resources/nls/Help.properties
b6eeb4e
@leonrohne27
Delete cli/src/main/java/com/devonfw/tools/ide/tool/squirrelsql/Squir…
b9d047b 
…relSqlUrlUpdater.java
@leonrohne27
Delete cli/src/main/java/com/devonfw/tools/ide/commandlet/CommandletM…
ab23635 
…anagerImpl.java
@leonrohne27
Delete cli/src/main/java/com/devonfw/tools/ide/tool/squirrelsql/Squir…
276bdda 
…relSql.java
@leonrohne27
Merge branch 'main' into fix/1130improve-behaviour-on-ambiguous-XPath…
71165cb 
…-match
@leonrohne27 leonrohne27 self-assigned this Mar 17, 2025
@leonrohne27 leonrohne27 deleted the fix/1130improve-behaviour-on-ambiguous-XPath-match branch March 17, 2025 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@leonrohne27 leonrohne27

Labels
None yet
Projects
IDEasy board
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

improve behaviour on ambiguous XPath match
1 participant
@leonrohne27