hellothanos

Dear Kevin,

After inspecting and with some support from the Polylang dev team it looks like the error is coming from this line:
https://github.com/pixelbart/helpful/blob/8511226c3a8d6d3235b6cc9a1d74fb1f4ed318fb/core/services/class-options.php#L39

As I was told “the options are serialised before updating them into WordPress database. WordPress already serializes so it ends up that its done twice and Polylang (as well as WPML, almost certainly) cannot read the keys as they are unserializing but remain with a serialized block to read.”

As per their suggestion removing the ‘maybe_serialize’ method resolves the issue.

I hope this helps!

Best,
Thanos

Got you and I agree. It looks like a Polylang issue. I’ll investigate and post here when I find a solution, in case other users have the same problem.

Thank you for your hard work and support.

Best,
Thanos

Hey @pixelbart

Thanks for getting back to me and for the info and file.

I see where the issue seems to be coming from now: the new wpml-config.xml file nests the options in a helpful_options key. Because the structure is different the strings seem to somehow disappear.

This is how the translations looked before the upgrade:
https://i.snipboard.io/4rBQf8.jpg

And this is how they look after the upgrade:
https://i.snipboard.io/T29M6s.jpg

I hope this helps. I’ll keep investigating too!

Best,
Thanos

Hi again,
just to say that I just tested again on another site and both changes happened when I upgraded the plugin to the latest version, 4.5.6.
I got the strings back by downgrading to 4.4.70 ??

Thanos

helpful_pre_save_vote was what I was looking for, thank you!
And thank you for the extra effort to suggest the Wp Rocket solution ??

Best,
Thanos

Hi Kevin,

Thank you for the quick response and for sorting this!

This works great – the only small issue is that the file is being downloaded with a .html extension. I’m guessing it’s because the Content-Type header of the file isn’t set to text/csv. I saw you’re using the wp_send_json function that sets the headers to application/json so not sure how you could go about this.

A workaround that I tested and seems to work ok on chrome is setting the a.download attribute’s value to log.csv. Apparently though you’re not supposed to put extensions to the value https://www.w3schools.com/tags/att_a_download.asp ??

Hope this helps,
Thanos

• This reply was modified 3 years ago by hellothanos.