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.

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

Double quotes in a va-link (Staff Name) field are not encoded properly for the FE #18193

Open
3 tasks
ian-sears opened this issue May 22, 2024 · 2 comments
Open
3 tasks
Labels
Defect Something isn't working (issue type) Facilities Facilities products (VAMC, Vet Center, etc) Needs refining Issue status sitewide VA.gov frontend CMS team practice area VAMC CMS managed product owned by Facilities team

Comments

@ian-sears
Copy link

ian-sears commented May 22, 2024

Describe the defect

When an editor uses double quotes in a Staff Profile first or last name field, this causes the name to not be displayed on the Front end

To Reproduce

Steps to reproduce the behavior:
Maybe do this on a Tugboat so we don't break this again on Prod:

  1. Go to a staff profile page, for example Charles "Dave" Collins
  2. Change the single quotes to double quotes for his nickname "Dave"
  3. Save the Node.
  4. After Content Release, the page at Salisbury Health Care > Leadership displays the name as only Charles.

AC / Expected behavior

  • If double quotes are used in the Drupal Staff field, these are encoded properly in the FE build
  • The entire First and Last Name content should display on the Front end
  • QA Review

Screenshots

I will try to capture a screenshot to add to this if possible.

Additional context

@ian-sears ian-sears added Defect Something isn't working (issue type) Needs refining Issue status labels May 22, 2024
@github-actions github-actions bot added the Facilities Facilities products (VAMC, Vet Center, etc) label May 22, 2024
@ian-sears ian-sears added Drupal engineering CMS team practice area accessibility Issues related to accessibility labels May 22, 2024
@laflannery laflannery added VA.gov frontend CMS team practice area VAMC CMS managed product owned by Facilities team and removed accessibility Issues related to accessibility Drupal engineering CMS team practice area labels May 22, 2024
@laflannery laflannery changed the title Double quotes in a Staff Profile prematurely end the text being pushed to the front end Double quotes in a Staff Name field are not encoded properly for the FE May 22, 2024
@jilladams jilladams changed the title Double quotes in a Staff Name field are not encoded properly for the FE Double quotes in a va-link (Staff Name) field are not encoded properly for the FE May 22, 2024
@srancour
Copy link
Contributor

Charles is no longer listed on that page. Another page to look at is Spokane Health Care Leadership with James "Shep" Speight

@srancour
Copy link
Contributor

They changed James to "James (Shep) Speight" another leadership member is Thomas "Zach" Orr who's on the VA West Palm Beach Health Care Leadership page. It does look like the name may being populated is from the text field in the va-link element because it is in fact closing that field with the quotes for this instance.

On the Sergeant Ernest I. "Boots" Thomas VA Clinic facility it looks like the text field is escaping the quotes using &quot on the Locations page.

Staff profiles being referenced in pages other than the leadership page seem to not be using the va-link and instead are using a tags so the quotes are working there for things like Mary "Heather" Levy BSN, RN on the Charleston Healthcare Oncology Meet the Team section.

Also, when a staff profile doesn't have a biography it seems that it doesn't break as well like in the case of Robert "Robbie" Davis on the VA North Florida/South Georgia health care Patient advocates page. This is also true for leadership pages when the person doesn't have a biography like William "Billy" Irwin MD on the VA Marion health care Leadership page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Something isn't working (issue type) Facilities Facilities products (VAMC, Vet Center, etc) Needs refining Issue status sitewide VA.gov frontend CMS team practice area VAMC CMS managed product owned by Facilities team
Projects
None yet
Development

No branches or pull requests

4 participants