`rex_media_plus::get("beispielbild.jpg")->getBackgroundStyles($group);` funzt nicht -> whooops

[ynamite]

echo rex_media_plus::get("beispielbild.jpg")->getBackgroundStyles($group); funktioniert nicht und gibt einen whooops.

In rex_media_plus.php habe ich die Methoe getBackgroundStyleswie folgt geändert:

public function getBackgroundStyles($group_name, $selector)
{
    return media_manager_type_group::getBackgroundStyles($this->getFilename(), $group_name, $selector);
}

und Aufruf dann wie folgt, dann klappts :)

echo rex_media_plus::get('REX_MEDIA[1]')->getBackgroundStyles('gruppenname', '#hintergrundbild');

Sehr cooles Addon btw.

[alxndr-w]

Tipp: Wenn du's direkt als PR machst, sehe ich den Unterschied und was geändert wurde. Jetzt ist es mühsam, die Datei aufzurufen, nebeneinander zu legen, zu vergleichen, usw.

Da ich den Teil in mehreren Projekten erfolgreich im Einsatz habe, wundert es mich, was nicht klappt. Was war denn Inhalt des Whooops?

[ynamite]

Da es nur 2 geänderte Zeilen sind, wollte ich nicht extra einen PR machen. Vor allem weil ich nicht sicher war, ob mein Fix in die richtige Richtung geht.

Meiner Meinung nach konnte das gar nicht funktionieren, weil die aufgerufene Methode Parameter erwartet, die nicht mitgegeben wurden. Das war auch der Whooops, expecting at least 2 parameters, none sent (oder so).

Kann aber sonst gerne noch einen PR nachreichen.

[ynamite]

Hier der PR: https://github.com/alexplusde/media_manager_responsive/pull/19

Und hier der Whooops:

ArgumentCountError: Too few arguments to function media_manager_type_group::getBackgroundStyles(), 0 passed in /redaxo/src/addons/media_manager_responsive/lib/rex_media_plus.php on line 65 and at least 3 expected File: redaxo/src/addons/media_manager_responsive/lib/media_manager_type_group.php Line: 92

Stacktrace

| Function | File | Line | | --------------------------------------------- | ------------------------------------------------------------------------ | -------- | | media_manager_type_group::getBackgroundStyles | redaxo/src/addons/media_manager_responsive/lib/rex_media_plus.php | 65 | | rex_media_plus->getBackgroundStyles | theme/private/template-default.php | 34 | | require | theme/private/template.php | 26 | | require | rex:///template/1 | 2 | | require | redaxo/src/addons/structure/plugins/content/lib/article_content_base.php | 460 | | rex_article_content_base->{closure} | redaxo/src/core/lib/util/timer.php | 57 | | rex_timer::measure | redaxo/src/addons/structure/plugins/content/lib/article_content_base.php | 457 | | rex_article_content_base->getArticleTemplate | redaxo/src/addons/structure/plugins/content/boot.php | 60 | | rex_package::{closure} | redaxo/src/core/lib/extension.php | 52 | | rex_extension::{closure} | redaxo/src/core/lib/util/timer.php | 57 | | rex_timer::measure | redaxo/src/core/lib/extension.php | 42 | | rex_extension::registerPoint | redaxo/src/core/frontend.php | 26 | | require | redaxo/src/core/boot.php | 143 | | require | index.php | 9 |

System report (REDAXO 5.14.2, PHP 8.2.2, MariaDB 10.3.37)

| REDAXO | | | ------------: | :--------- | | Version | 5.14.2 | | PHP | | | ------------: | :--------- | | Version | 8.2.2 | | OPcache | no | | Xdebug | no | | Database | | | ------------: | :-------------- | | Version | MariaDB 10.3.37 | | Character set | utf8mb4 | | Server | | | ------------: | :--------- | | OS | Linux | | SAPI | cgi-fcgi | | Webserver | Apache | | Request | | | ------------: | :------------ | | Browser | Firefox/109.0 | | Protocol | HTTP/1.1 | | HTTPS | yes | | Packages | | | ------------------------------: | :---------- | | adminer | 1.9.3 | | backup | 2.8.2 | | be_password | 2.0.1 | | be_style | 3.1.1 | | be_style/customizer | 3.1.0 | | be_style/redaxo | 3.1.1 | | be_tools | 1.7.8 | | be_tools/informations | 1.0 | | bloecks | 3.1.1 | | bloecks/cutncopy | 3.1.1 | | bloecks/dragndrop | 3.1.1 | | bloecks/status | 3.1.1 | | cache_warm_up | 1.1.0-dev | | cropper | 1.3.0 | | developer | 3.9.2 | | emailobfuscator | 3.0.0 | | flatpickr | 1.1.0 | | focuspoint | 4.0.3 | | install | 2.10.0 | | massif_minify | 1.3.2 | | media_manager | 2.13.2 | | media_manager_responsive | 1.1.0 | | mediapool | 2.12.0 | | metainfo | 2.9.0 | | phpmailer | 2.11.2 | | project | dev | | quick_navigation | 6.0.2 | | seocheckup | 1.6.6 | | speed_up | 1.3.1 | | sprog | 1.5.1 | | structure | 2.14.2 | | structure/content | 2.14.2 | | structure/history | 2.14.2 | | theme | 1.3.3 | | theme_lock | 1.0.0 | | tinymce5 | 1.2.16 | | uploader | 2.3.0 | | url | 2.1.0 | | useragent | 2.3.0 | | users | 2.9.1 | | yform | 4.0.4 | | yform/email | 4.0.4 | | yform/manager | 4.0.4 | | yform/tools | 4.0.4 | | yform_quick_edit | 1.0.1 | | yform_spam_protection | 1.2.3 | | yform_spam_protection/recaptcha | 1.0.0-beta1 | | yform_usability | 2.0.7 | | yrewrite | 2.9.1 |

[alxndr-w]

Scheint schlüssig, ich hatte das wohl genau so vorgesehen aber dann an der Stelle nicht weiterentwickelt.