There are legitimate reasons to have the rewrite fieldset, especially the options for stripping tags and whitespace.

CommentFileSizeAuthor
#2 do-not-disable-rewrite-8-x-3053211-2.patch365 bytesandileco

Comments

andileco created an issue. See original summary.

andileco’s picture

andileco commented
Related issues: +#2310677: Do not disable rewrite
StatusFileSize
new do-not-disable-rewrite-8-x-3053211-2.patch365 bytes

Patch attached.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Status: Active » Needs review

+1

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Status: Needs review » Reviewed & tested by the community

Applies and works. Thank you 🍺

brad.bulger’s picture

brad.bulger commented

+1 on adding this. I needed to wrap the included view output in custom HTML that included values from other fields in the outer view, but not if the included view output is empty. The only way to do that is to do the custom HTML in the view field. This patch applied to 1.0.0-beta5 with only some whitespace warnings.

ronaldmulero’s picture

ronaldmulero commented

Works. Needed. Thank you.
Drupal 10.2.x
Views Field View 8.x-1.0-beta5

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Status: Reviewed & tested by the community » Fixed

Thank you for your contributions.
Merged. 🍻

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • vladimiraus committed e052cd59 on 8.x-1.x 
    feat: #3053211 Do not disable rewrite (8.x)

By: andileco
By:...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.