- 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
Please use an issue fork and merge request.
- 🇮🇳India prem suthar Ahemdabad- Gujrat , Jodhpur - Rajsthan
prem suthar → made their first commit to this issue’s fork.
In some situations, it seems that the fix for #3020601 introduces a data compatibility issue.
Briefly, we expect the data to be something like
"field_facility_service_hours": [
{
"value": [
[
"Mon",
""
],
[
"Tue",
""
],
But in some cases it will present as:
"field_facility_service_hours": [
{
"value": {
"0": [
"Mon",
"8:00 a.m. to 4:30 p.m. ET"
],
"caption": "",
"1": [
"Tue",
"8:00 a.m. to 4:30 p.m. ET"
],
The issue is the insertion of the `caption` key, which causes the value to be JSON-encoded as an object instead of an array.
Looking in the database, we see the serialized values for some rows being
a:7:{i:0;a:2:{i:0;s:4:"Mon";i:1;s:26:"8:00 a.m. to 4:30 p.m. ET";}i:1;a:2:{i:0;s:4:"Tue";i:1;s:26:"8:00 a.m. to 4:30 p.m. ET";}i:2;a:2:{i:0;s:4:"Wed";i:1;s:26:"8:00 a.m. to 4:30 p.m. ET";}i:3;a:2:{i:0;s:4:"Thu";i:1;s:26:"8:00 a.m. to 4:30 p.m. ET";}i:4;a:2:{i:0;s:4:"Fri";i:1;s:26:"8:00 a.m. to 4:30 p.m. ET";}i:5;a:2:{i:0;s:4:"Sat";i:1;s:0:"";}i:6;a:2:{i:0;s:4:"Sun";i:1;s:0:"";}}
and others being
a:8:{i:0;a:2:{i:0;s:3:"Mon";i:1;s:25:"8:00 a.m. to 4:30 p.m. ET";}s:7:"caption";s:0:"";i:1;a:2:{i:0;s:3:"Tue";i:1;s:25:"8:00 a.m. to 4:30 p.m. ET";}i:2;a:2:{i:0;s:3:"Wed";i:1;s:25:"8:00 a.m. to 4:30 p.m. ET";}i:3;a:2:{i:0;s:3:"Thu";i:1;s:25:"8:00 a.m. to 4:30 p.m. ET";}i:4;a:2:{i:0;s:3:"Fri";i:1;s:25:"8:00 a.m. to 4:30 p.m. ET";}i:5;a:2:{i:0;s:3:"Sat";i:1;s:6:"Closed";}i:6;a:2:{i:0;s:3:"Sun";i:1;s:6:"Closed";}}
This causes consistency issues for the consumers of our content export.
I'm not sure if this is the best way to approach this issue, but I'll submit a patch that removes the problematic lines.
Needs work
3.0
Code
The patch will have to be re-rolled with new suggestions/changes described in the comments in the issue.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Please use an issue fork and merge request.
prem suthar → made their first commit to this issue’s fork.