simplesamlphp / simplesamlphp/simplesamlphp-module-metarefresh
writeMetadataPdo surplus configuration
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 9
- Forks
- 19
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
There's a Too many arguments to function __construct(). 2 provided, but 1 accepted. warning being picked up in MetaLoader.php:675
The constructor on Metadata\MetaDataStorageHandlerPdo accepts one config parameter but then doesn't do anything with it anyway, calling Database::getInstance(); instead.
My inclination was to drop passing the config in the metarefresh module entirely but not sure if that's the right approach.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne bei MetaLoader.php:675 und untersuche die Konstruktion von Metadata\MetaDataStorageHandlerPdo. Vergleiche sie anschließend mit dem Konstruktor dieser Klasse und dessen Verwendung von Database::getInstance(). Ermittle, ob die überzählige Konfiguration entfernt oder konsistent behandelt werden sollte, und überprüfe, dass die Warnung des Konstruktors nicht mehr erscheint.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- database
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100