Skip to content

Properly encode filter string in feature editor#1957

Merged
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:properly_encode_filter
Sep 4, 2025
Merged

Properly encode filter string in feature editor#1957
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:properly_encode_filter

Conversation

@laeubi
Copy link
Contributor

@laeubi laeubi commented Sep 4, 2025

Currently if one uses a filter in a feature import that happens to use special characters like '&' that require encoding as an entity saving the feature will break the XML as it will write out the literal value but not the encoded entity. One then has to manually fix up the results.

This now uses the PDEXMLHelper.getWritableAttributeString to properly encode this as and attribute value before writing.

Currently if one uses a filter in a feature import that happens to use
special characters like '&' that require encoding as an entity saving
the feature will break the XML as it will write out the literal value
but not the encoded entity. One then has to manually fix up the results.

This now uses the PDEXMLHelper.getWritableAttributeString to properly
encode this as and attribute value before writing.
@github-actions
Copy link

github-actions bot commented Sep 4, 2025

Test Results

   765 files  ±0     765 suites  ±0   47m 55s ⏱️ - 1m 8s
 3 611 tests ±0   3 557 ✅ +1   54 💤 ±0  0 ❌  - 1 
10 833 runs  ±0  10 670 ✅ +1  163 💤 ±0  0 ❌  - 1 

Results for commit 41cb256. ± Comparison against base commit 6f0d306.

@laeubi laeubi merged commit 0a4c726 into eclipse-pde:master Sep 4, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant