Plugin BROKE with HTTPS://

  • Resolved Richard

    (@rckelsey)


    7 years, 8 months ago

    Hi,

    will there be an update for the plugin anytime soon? To accommodate SSL sites?

    The plugin DOES NOT WORK with https, and we are changing all of our sites to this, AND we have lots of bands that rely on Bands in Town, for their shows to show up on their sites.

    Any help or update would be appreciated.

    Thank you!

    Best,
    Richard

    • This topic was modified 7 years, 8 months ago by Richard.
Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Bandsintown

    (@kwestion505)

    Hi Richard, the latest version 1.1.1. addresses this issue.

    Hi Phil,
    Thanks for the update, getting this (Stack Overflow: The called constructor method for WP_Widget is deprecated since version 4.3.0) , after updating to 1.1.2 and fixing/adding data attributes that changed or were added to v2 API. Can verify that an identical plugin with parent::__construct in place of parent::WP_Widget in bandsintown.php on line 178 as Stack Overflow answer recommends solves the issue. Any chance of adding the source to @bandsintown GH? I would add a PR there if you want, but know that is asking a lot so if it’s easier to fix yourself and leave SVN as is, totally understand.

    Ricky

    • This reply was modified 7 years, 1 month ago by Ricky de Laveaga. Reason: typo

    make that line 175 in bandsintown.php with parent::__construct in place of parent::WP_Widget

    Plugin Author Bandsintown

    (@kwestion505)

    Hi Ricky, this should be addressed with the latest version 1.1.3. Thanks for reporting the issue.

    Thanks so much for the lightning fix. I noticed in testing the patched version that in the minified JS the function that sets the eventPopupURL later used to open the window is setting var a to "https://tourdates.bandsintown.com/event/"+this.id;return g(a).setQuery(this.collection.eventPopupQueryParams()).toString() which does not return events for me where "https://bandsintown.com/event/"+… does. Removing tourdates. has events and the Tickets and More links returning the event. Thinking you would probably be editing unminified original file I checked to see if that was available and it was, thanks for that, this is line 25156 in main.js.

    Once I had event links working on new API, I noticed a separate problem later on in the JS (line 25479 in main.js)…
    this.popupWindow=e.openWindow(a,b,this.popupOptions(375,627))

    …Creates an unusable cropped popup window (page layout does not respond to small viewport) where passing a version minus ,this.popupOptions(375,627)…
    this.popupWindow=e.openWindow(a,b)

    …Spawns a full window in a new tab as I would expect and I imagine most downstream consumers of the plugin would also. Thanks again for the attention, and if I can help by committing these proposed changes to the JS somewhere, please let me know.

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Plugin BROKE with HTTPS://’ is closed to new replies.