Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.9.0] System - Privacy Consent - Modal with Privacy Article requires MooTools #22908

Closed
jwiesel opened this issue Nov 1, 2018 · 3 comments

Comments

Projects
None yet
5 participants
@jwiesel
Copy link

commented Nov 1, 2018

Steps to reproduce the issue

  • Disable MooTools on your site (e.g. using plugin "MooTools Enabler/Disabler"
  • Activate plugin "System - Privacy Consent"
  • In the plugin options set a "Privacy Article"
  • With a user without a valid consent try to login
  • Below "Data Privacy", hover over or click on the star icon with the privacy policy

Expected result

  • A modal dialog should open showing the data privacy article

Actual result

  • No modal window appears. Actually nothing happens except showing the popover
  • Developer console of the web-browser shows two errors

ReferenceError: Hash is not defined
TypeError: SqueezeBox is undefined

System information (as much as possible)

  • Joomla 3.9.0
  • PHP 7.2

Additional comments

  • Can we use Bootstrap modals instead of MooTools for the plugin??
@zero-24

This comment has been minimized.

Copy link
Contributor

commented Nov 2, 2018

Please check & test this one: #22926 as this is the first partial fix of this issue touching the Content ConfirmConsent Plugin. When approved similar code can be applied to the other places you mention.

@brianteeman brianteeman added the J3 Issue label Nov 6, 2018

@joomla-cms-bot

This comment has been minimized.

Copy link

commented Mar 4, 2019

@franz-wohlkoenig

This comment has been minimized.

Copy link
Member

commented Mar 4, 2019

closed as having Pull Request #22926

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.