search

FriendsOfREDAXO / yform_usability

Ergänzt YForm um nützliche Funktionen wie Drag & Drop und Status On/Off.
MIT License
42 stars 3 forks source link

addStatusToggle return type auf yform4 angepasst #132

Closed michael-kreatif closed 1 year ago

michael-kreatif commented 1 year ago

Rückgabe kann auch rex_yform_list sein, nicht nur rex_list. Siehe Code Stellen:

https://github.com/yakamara/redaxo_yform/blob/ced8f147a76caae745281fa112b3f1a17a807efd/plugins/manager/fragments/yform/manager/page/list.php#L16 https://github.com/yakamara/redaxo_yform/blob/ced8f147a76caae745281fa112b3f1a17a807efd/plugins/manager/fragments/yform/manager/page/list.php#L175 https://github.com/FriendsOfREDAXO/yform_usability/blob/main/boot.php#L65

tyrant88 commented 1 year ago

Es ist eigentlich immer eine rex_yform_list würde ich behaupten. Weiß gerade nicht, warum das bei mir keinen Fehler geworfen hat...

michael-kreatif commented 1 year ago

@tyrant88 Stimmt, danke, laut Code müsste es eigentlich immer rex_yform_list sein, außer es gibt noch ein anderes addon, das den EP 'YFORM_DATA_LIST' triggert, aber das glaube ich nicht.

tyrant88 commented 1 year ago

Ich glaube das widerspräche dem Konzept der Extension Points.

tyrant88 commented 1 year ago

Ah, sehe gerade im yform changelog für Version 4: "rex_yform_list ergänzt. Ersetzt rex_list und kann YORM Queries entgegennehmen" Deshalb hatte Thomas Blum wohl rex_list im Code...

goldfoot commented 1 year ago

Typsicherheit ist ja ansich nicht schlecht und in PHP 8 sind Union Types erlaubt. Ich schlage daher vor, beide Typen zuzulassen und den PR in "addStatusToggle return type rex_yform_list zulassen" umzubenennen:

protected static function addStatusToggle($list, $table): \rex_list | \rex_yform_list
    {
        /** @var \rex_list | \rex_yform_list $list */
       // ...
    }
tyrant88 commented 1 year ago

@goldfoot ? Wir haben doch bereits geklärt, dass ab yform 4 (was in der package.yml vorrausgesetzt wird) der return type "rex_yform_list" ist.

Eventuell merge ich den PR so wie er ist, wenn der Ersteller ihn nicht zeitnah anpasst, deshalb bleibt der Titel erstmal. :-)

goldfoot commented 1 year ago

@tyrant88 Stimmt, ich hatte die Beschränkung der yForm Version nicht berücksichtigt und bin daher davon ausgegangen, dass auch yForm 3 noch möglich sei.