TypeError: Argument 1 passed to StudipCachedArray::loadData() must be of the type string, null given

Beschreibung

[Tue Jan 25 11:05:29.255991 2022] [php7:notice] [pid 9367] [client 84.118.82.60:44268] TypeError: Argument 1 passed to StudipCachedArray::loadData() must be of the type string, null given, called in /home/studip/studip-git/lib/classes/StudipCachedArray.php on line 53 and defined in /home/studip/studip-git/lib/classes/StudipCachedArray.php:104\nStack trace:
#0 /home/studip/studip-git/lib/classes/StudipCachedArray.php(53): StudipCachedArray->loadData(NULL)
#1 /home/studip/studip-git/lib/plugins/db/RolePersistence.class.php(220): StudipCachedArray->offsetExists(NULL)
#2 /home/studip/studip-git/lib/plugins/db/RolePersistence.class.php(206): RolePersistence::loadUserRoles(NULL, true)
#3 /home/studip/studip-git/public/plugins_packages/studip/PluginMarket/classes/MarketPluginUsage.php(38): RolePersistence::isAssignedRole(NULL, 'Pluginbeauftrag...')
#4 /home/studip/studip-git/public/plugins_packages/studip/PluginMarket/views/presenting/details.php(58): MarketPluginUsage->isEditable()
#5 /home/studip/studip-git/vendor/flexi/lib/php_template.php(46): require('/home/studip/st...')
#6 /home/studip/studip-git/vendor/flexi/lib/template.php(157): Flexi_PhpTemplate->_render()
#7 /home/studip/studip-git/vendor/trails/trails.php(764): Flexi_Template->render()
#8 /home/studip/studip-git/vendor/trails/trails.php(724): Trails_Controller->render_template('presenting/deta...', Object(Flexi_PhpTemplate))
#9 /home/studip/studip-git/vendor/trails/trails.php(564): Trails_Controller->render_action('details')
#10 /home/studip/studip-git/app/controllers/studip_controller.php(110): Trails_Controller->perform('details/b981b98...')
#11 /home/studip/studip-git/vendor/trails/trails.php(154): StudipController->perform('details/b981b98...')
#12 /home/studip/studip-git/vendor/trails/trails.php(121): Trails_Dispatcher->map_uri_to_response('presenting/deta...')
#13 /home/studip/studip-git/lib/plugins/core/StudIPPlugin.class.php(166): Trails_Dispatcher->dispatch('presenting/deta...')
#14 /home/studip/studip-git/public/plugins.php(54): StudIPPlugin->perform('presenting/deta...')
#15 {main}, referer: https://develop.studip.de/studip/plugins.php/pluginmarket/presenting/all?search=Etherpad

added BIEST Version5.0 labels

assigned to @tleilax

closed with commit b97297c6

mentioned in commit 0d78456c

changed milestone to %Stud.IP 5.0

mentioned in commit b97297c6

Ist das nicht eher ein Fehler im Plugin? Ich denke nicht, daß man isAssignedRole() ohne user id aufrufen darf.

Ja, es ist eigentlich ein Fehler im Plugin, der von Stud.IP vorher aber stillschweigend akzeptiert wurde. Da ich aber vermute, dass der Fehler genauso in anderen Plugins auftreten könnte, finde ich es nicht verkehrt, das Problem auch in RolePersistence::isAssignedRole() abzufangen.

Wäre es dann nicht besser, wenn so ein Aufruf dann direkt auf einen Fehler läuft?

Theoretisch schon, aber dann mache ich mit einem BIEST potentiell Dinge kaputt, die vorher funktioniert haben. Kann ich machen, hatte mich aber doch für die etwas vorsichtigere Variante entschieden.

Wenn Du meinst, das sollte besser kaputtgehen, kann ich das gerne wieder zurücknehmen.

Meine Meinung - etwas abstrakter formuliert - dazu ist:

Wenn ein Plugin (oder auch Code im Kern) eine API falsch benutzt, sollte das dem Entwickler auf die Füße fallen. Falsche Parameter stillschweigend zu akzeptieren, führt auf Dauer nur dazu, daß Fehler unbemerkt bleiben - wir hatten das ja schon bei Davids Umbauten an der Deputies-API, wo auch viele Stellen über Jahre falsche Werte übergeben hatten und es dann plötzlich auffiel, als das zu einem Fehler führte. Man kann jetzt darüber streiten, wann der richtige Zeitpunkt ist, hier die Parameter besser zu prüfen - das hängt aber auch maßgeblich von der Anzahl der betroffenen Plugins ab (aktuell nur eins).

In diesem Konkreten Fall ist mir das relativ egal, weil die API dieser Klasse eh grottig ist und komplett überarbeitet werden müßte.

Gut, dann lasse ich das erstmal so und wenn wir das Refaktorisieren sollten, können wir den korrekten Typ ja erzwingen.

mentioned in commit e16e59cd