Problem/Motivation

If view contains a non-numeric field (for example group by state in address field) template doesn't sufficiently clean up string value to be used as id.

Steps to reproduce

- content type contains address field (with state)
- create view display items grouped by state
- template is broken

Proposed resolution

- clean up key to be usable as div id

Example: see attached screenshot.

CommentFileSizeAuthor
#5 render-issue.png133.67 KBjannakha

Issue fork views_bootstrap-3573315

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jannakha created an issue. See original summary.

jannakha opened merge request !57

jannakha’s picture

jannakha commented
Title: Accordion template breaks for strings as header/group » Accordion template breaks if header group is a string or rendered entity
jannakha’s picture

jannakha commented
Status: Active » Needs review

please review MR57

jannakha’s picture

jannakha commented
StatusFileSize
new render-issue.png133.67 KB
vladimiraus’s picture

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

Looks good. Applied and tested.
Thank you for your contribution. RTBC!

  • shelane committed beb0b599 on 5.5.x authored by jannakha 
    Issue #3573315 : Accordion template breaks if header group is a string...
shelane’s picture

shelane
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

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.

Status: Fixed » Closed (fixed)

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