- Issue created by @kobusvw
- 🇮🇳India joshua1234511 Goa
Issue fixed in https://github.com/civictheme/uikit/pull/346
- 🇦🇺Australia RichardGaunt Melbourne
The fix updates the parsing of the incorrect "plain text" from the API but the root cause of this issue is in the `plain_text` field formatter which adds the twig filter `nlbr` to its processing.
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/lib/Drupal/Co...
The ideal fix will be to change the page visibility field to a multi-value plain text field so it provides an array of page visibility values and we can iterate over the values.
Due to this requiring a new field / update existing field and an upgrade path this will need to be considered.
For the meantime Josh's fix mitigates the issue
- Assigned to RichardGaunt
- 🇦🇺Australia RichardGaunt Melbourne
Sorry on further inspection I see that we are using a REST display of alert content types see: /admin/structure/views/view/civictheme_alerts
The `visibility` field needs to be updated to strip tags to remove the `
` tags.Same underlying cause but we have ability to change field output.
- Assigned to joshua1234511
- Assigned to RichardGaunt
- Status changed to Needs review
5 months ago 5:15am 4 September 2024 - 🇦🇺Australia RichardGaunt Melbourne
https://github.com/civictheme/uikit/pull/346
I have left a few questions, Josh - Status changed to Needs work
5 months ago 8:51am 4 September 2024 - Status changed to Needs review
4 months ago 10:59pm 16 September 2024 - Status changed to Fixed
4 months ago 3:10pm 19 September 2024 - 🇮🇳India sonam.chaturvedi Pune
Verified and tested on 1.8-dev
Testing Results:
When we set Page visibility condition on Alert content type then validation does not fail for any urls listed - PASSScreenshot:
- Status changed to RTBC
4 months ago 4:01pm 19 September 2024 Automatically closed - issue fixed for 2 weeks with no activity.
@richard noting this issue closed via automation. Kobus tells me this is not in the code base. I will need to check on other tickets that have automatically closed.
Automatically closed - issue fixed for 2 weeks with no activity.