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

Mblock 3.1.0 - Widgets in REDAXO 5.12.1 defekt #137

Closed
skerbis opened this issue Jun 21, 2021 · 5 comments
Closed

Mblock 3.1.0 - Widgets in REDAXO 5.12.1 defekt #137

skerbis opened this issue Jun 21, 2021 · 5 comments
Assignees
Labels

Comments

@skerbis
Copy link
Member

skerbis commented Jun 21, 2021

Die Medienauswahl funktioniert nicht mehr.

geändert haben sich die onclick attribute
onclick="addREXMedia(1, '...')" etc.
da sind jetzt neu anführungszeichen um die id vorne.
yform hat den js-code in onclick per regex angepasst, und das passte daher nicht mehr.
weiß nicht, ob das vielleicht auch in mblock das problem ist

Gregor hat folgenden Lösungsvorschlag geliefert:

für die Zeilen 172 und 195 in mblock.js

$(this).attr('onclick', $(this).attr('onclick').replace(/(\('?)\d+/, '$1' + sindex + '' + mblock_count + '00' + eindex));

Das funktioniert ab dem 2. Block.

in YForm wurde es wie folgt gelöst:

yakamara/yform@ce63947

@skerbis skerbis added the bug label Jun 21, 2021
@skerbis skerbis changed the title Mblock Widgets in 5.12.1 Mblock Widgets in 5.12.1 defekt Jun 21, 2021
@skerbis skerbis changed the title Mblock Widgets in 5.12.1 defekt Mblock 3.1.0 - Widgets in REDAXO 5.12.1 defekt Jun 21, 2021
@joachimdoerr
Copy link
Member

0a7c570

Das Problem sollte mit dem Commit behoben sein.

@shauste
Copy link
Member

shauste commented Jun 24, 2021

Konnte das Problem leider nicht lösen durch das Abändern der o.g. Zeilen – „Datei übernehmen“ ist nicht möglich.

Ausgabe Konsole:
Uncaught TypeError: Cannot set property 'value' of null at selectMedia (mediapool.js?buster=1624340028:232) at HTMLAnchorElement.onclick (index.php?page=mediapool/media:245)

@joachimdoerr
Copy link
Member

Hier fehlte der PHP Part für die Initialvergabe der oncklick tags in den Widgets, das JS replacement greift nur im Falle der Duplizierung. Mit dem letzten Commit sollen nun alle System-Widgets wieder laufen. 77d83f6

@shauste
Copy link
Member

shauste commented Jun 25, 2021

@joachimdoerr Konnte ich testen, klappt.

@steri-rex
Copy link

danke - hat bei mir auch geklappt!

@skerbis skerbis closed this as completed Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants