wpDiscuz - WordPress Comment Plugin
Share:
Notifications
Clear all

Compatibility problems with WpDiscuz plugin

3 Posts
2 Users
0 Likes
92 Views
P.A.Semi
Posts: 1
Topic starter
(@p-a-semi)
New Member
Joined: 2 months ago

On Firefox 45 there is error: document.querySelectorAll().forEach is not a function.

A simple polyfill fix is:

[code]
(function(){function I(O,p){if(p=Object.getPrototypeOf(O))p.forEach||(p.forEach=[].forEach)}try{I(document.querySelectorAll("body"));I(document.getElementsByClassName("site"))}catch(E){console&&console.error(E)}})();
[/code]

Clpx109103 crop

I've added it in class.WpdiscuzCore.php in frontendFiles function, but it was overwritten in next wpdiscuz upgrade, so I had to block upgrades temporarily...

Another problem in this version of Firefox is with padding:2% on input fields in /wp-content/plugins/wpdiscuz/themes/default/style.css?ver=7.3.19, which makes them completely unusable, because it calculates 2% from page height, and text has few pixels at center of line hiding all input...

Clpx109101 crop2b

 

And just a side-note, that bubble updates are very bad idea on web-site with 10000+ daily readers, it is not fast (on that server with slow database it was cca 1 second server-time per request) and it collapsed our server, when the wp-json request has been unblocked by provider... I already switched that off (removed in script and disabled in configuration) three weeks ago, but until this day multiple tens of users have opened pages from May and still bomb our server with /wp-json/wpdiscuz/v1/update? requests... I had to add filter at start of main index.php to reply them with fast 400... (so for now there is no more problem with that)

2 Replies
Asti
Posts: 5091
 Asti
Support
(@asti)
Illustrious Member
Joined: 5 years ago

We're going to check this. Please wait a bit. I'll update this topic ASAP. 

Reply
1 Reply
Asti
 Asti
Support
(@asti)
Joined: 5 years ago

Illustrious Member
Posts: 5091

This will be fixed in the next version. 

Reply
Share: