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

rex_input::factory für Widgets #24

Closed
cukabeka opened this issue Oct 25, 2017 · 8 comments
Closed

rex_input::factory für Widgets #24

cukabeka opened this issue Oct 25, 2017 · 8 comments

Comments

@cukabeka
Copy link
Member

// Dateiauswahl Medienpool-Widget

Könnte man nicht hier mit der Factory arbeiten, um das weiter zu vereinfachen?

Beispiel aus R4:

$id = 1;                                                   // ID dieses MediaListbuttons
$mp = 4;                                                   // ID der auzurufenden Medienpool Kategorie
$tmp = rex_input::factory('medialistbutton');              // Objekt initialisieren
$tmp->setButtonId($id);                                    // Button ID
$tmp->setCategoryId($mp);                                  // Medienpool Kategorie ID
$tmp->setValue($myREX['settings']['MEDIALIST'][$id]);      // gespeicherte Werte einsetzen
$tmp->setAttribute('name', 'MEDIALIST['.$id.']');
$MediaList1 = $tmp->getHtml();
@gharlan
Copy link
Member

gharlan commented Oct 25, 2017

rex_input ist eine interne Klasse aus dem metainfo-Addon:
https://github.com/redaxo/redaxo/blob/master/redaxo/src/addons/metainfo/lib/input.php#L6
Da sie als internal markiert ist, sollte man sich nicht auf sie verlassen.
(Sie ist internal, da deren Zukunft noch nicht geklärt ist, eigentlich hat der core ja selbst schon formularelemente etc.)

Die zurzeit offizielle Variante, um eine Medialist selbst zu erstellen (außerhalb von Metainfos, rex_form etc.), sieht so aus:

$MediaList1 = rex_var_medialist::getWidget($id, $name, $value, $args);

https://github.com/redaxo/redaxo/blob/63dfbc8d7b7bdf256fd1309e76ddb12c588c6915/redaxo/src/addons/mediapool/lib/var_medialist.php#L44

@cukabeka
Copy link
Member Author

ah, danke für die info!

generell scheint das mit rex_form zu gehen, das wäre dann mit #25 abgedeckt - leider gibts da wiederum noch kein Beispiel für rex_form kombiniert mit rex_fragments..

@skerbis
Copy link
Member

skerbis commented Nov 29, 2017

Wir bekommen es auch nicht in einer Config-Seite hin eine Medialist zu erstellen. Wie sollte dort der korrekte Code aussehen?

@skerbis skerbis reopened this Nov 29, 2017
@aeberhard
Copy link
Member

Thomas, vielleicht so!?
https://github.com/FriendsOfREDAXO/demo_addon/blob/master/pages/config.php#L88-L118
demo_addon ist auch im Installer zum ausprobieren!

@skerbis
Copy link
Member

skerbis commented Nov 29, 2017

Jupp, das ist klar. Aber das muss doch einfacher gehen. Am besten wäre wenn man rex_form auch für config verwenden könnte. @aeberhard 🙏 danke

@aeberhard
Copy link
Member

Ah Thomas, verstehe schon.
Hab ein neues Issue aufgemacht da rex_input::factory ja nicht verwendet werden kann ...

@staabm
Copy link
Member

staabm commented Nov 30, 2017

wo ist das neue issue? bitte verlinken

@staabm
Copy link
Member

staabm commented Nov 30, 2017

ahh, habs gefunden #29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants