Formatter is giving Serialized php json data for meta tags. Need to fix that.

CommentFileSizeAuthor
#6 interdiff-2628610-5-6.txt1.75 KBheykarthikwithu
#6 2628610-6.patch1.86 KBheykarthikwithu
#5 interdiff-2628610-2-5.txt972 bytesaditya_anurag
#5 2628610-5.patch1.04 KBaditya_anurag
#2 2628610-2.patch686 bytesaditya_anurag

Comments

krknth created an issue. See original summary.

aditya_anurag’s picture

aditya_anurag commented
StatusFileSize
new 2628610-2.patch686 bytes

Please review the patch.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Status: Active » Needs work

Tested the patch, the meta data is showing up in the below format.
[{"field_meta":"{"title":"title ","description":"desc","abstract":"abssss","keywords":"kart, karthik, kumar"}"}]

aditya_anurag’s picture

aditya_anurag commented
Assigned: Unassigned » aditya_anurag
aditya_anurag’s picture

aditya_anurag commented
Assigned: aditya_anurag » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 2628610-5.patch1.04 KB
new interdiff-2628610-2-5.txt972 bytes

@heykarthikwithu,
thanks for reporting the issue.

We handled html markup as we need metatag as plain text.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
StatusFileSize
new 2628610-6.patch1.86 KB
new interdiff-2628610-5-6.txt1.75 KB

We have serialized data in the metatag value, and we will be converting the serialized data into the json data.
And this json data will be rendered in the twig file.
So instead of handling html markup at field formatter, we need the read the raw value from it in the twig file.
this is been implemented in the patch.

krknth’s picture

krknth commented
Status: Needs review » Reviewed & tested by the community

Looks good to me

  • heykarthikwithu committed 4f18eb9 on 8.x-1.x 
    Issue #2628610 by aditya_anurag, heykarthikwithu, krknth: Serialized...
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Status: Reviewed & tested by the community » Fixed
rakesh.gectcr’s picture

rakesh.gectcr
he/him
Primary language English
Location Manchester
commented
Status: Fixed » Needs work

I need little more workaround with this.

rakesh.gectcr’s picture

rakesh.gectcr
he/him
Primary language English
Location Manchester
commented
Status: Needs work » Closed (fixed)