TypeError: $ is not a function

Hey Denis!

Thanks for using Juicer that long and for putting up with this. ??

I’ll share this with our engineering team and keep you posted on the feedback and progress.

Meanwhile, have a great rest of the week!

Great, thanks!

Another issue this plugin has is that it loads juicer/includes/admin/js/admin.js on ALL pages in the backend(admin) which causes compatibility issues and errors like

Uncaught TypeError: Cannot read properties of null (reading 'addEventListener')
at HTMLDocument.<anonymous> (admin.js?ver=1.12.5:44:7)

I’d recommend loading this script ONLY on the admin.php?page=juicer-settings page.

Hope it makes sense.

Hi Denis,

Our devs are working on releasing Feeds 2.0 (it should go live towards the end of this summer), and these issues will not be present once that update goes live.

Please bear with us and these errors in the console while we work towards building better feeds. ??

Hi Denis,

Thank you for your patience while our engineers worked on this second part of your report.

I’m happy to say that your suggestion to load the admin script only on the admin.php page has been accepted and this has been fixed in our updated plugin. Please make sure you have the latest version installed.

If you don’t mind, I’ll mark this issue as resolved, as we will not be using jQuery with the upcoming feed changes – that will resolve the issue. Our engineers decided it’s better to focus on finishing the updated feeds rather than fixing something that we plan to replace next ??

Let me know if you have any other issues.

Have a great rest of the week!

Best,
Mario

P.S. If you prefer, you can also reach us at [email protected].

I still see this error:

Uncaught TypeError: $ is not a function
    at r.setFeedItemHeight (embed-no-jquery.js:18:3313)
    at o.<anonymous> (embed-no-jquery.js:18:493)
    at r.emitEvent (embed-no-jquery.js:2:24416)
    at r.emit (embed-no-jquery.js:2:24613)
    at embed-no-jquery.js:2:27805

It happens because of this code:

$(".words > .j-text").each((function() {
                return $(this).css("display", "none"),
                $(this).height(),
                $(this).css("display", "")
            }

You may want to replace this part $(".words > .j-text"). => jQuery(".words > .j-text").

So easy change. It’s been there for MONTHS of procrastination…

Hey Denis @mstrdh,

Our developers fixed this issue, you should not be encountering it anymore. I’m sorry it took so long, but thank you for bringing it up and for providing a quick fix for it.

Please let me know if there’s anything else you’d like me to bring up to our devs.

Best,
Mario