• MGChecker's avatar
    Umstellung der Klassenarchitektur und der Einstellungen auf JSON · 1f070179
    MGChecker authored
    Summary:
    Klassenarchitektur
    * Sichtbarkeit der Klassenattribute eingeschränkt, teilweise durch Getter und Setter weiterhin zugänglich gemacht
    * Statische Funktionen als solche markiert
    * Sichtbarkeit von httpRequest erweitert, das im Rahmen von MultiCore instanziert benötigt werden wird
    * Sichtbarkeit von login eingeschränkt, da es nur von initCurl aufgerufen werden soll
    * Finalisierung der beiden zentralen Funktionen httpRequest und initcurl, um Überschreiben zu verhindern
    * __construct als Alias zu initcurl für spätere Instanzierung im Rahmen von MultiCore
    
    Einstellungen via JSON
    * Einstellungen werden nun aus einem besser lesbaren JSON-File extrahiert, was neben den Login-Daten auch weitere Daten enthält, weswegen einige Parameter hinsichtlich https usw. entfallen
    * E-Mail jetzt nicht mehr per Account, sondern per Cygnus-Framework global
    * Kein Zwang, so die Benutzerdaten einzulesen, Kindklassen können andere Methoden verwenden
    
    Entfernen obsoleter Funktionen
    * addMail und sendMail erfüllen keinen Mehrwert gegenüber der internen Funktion mail()
    * initCurlArgs ist eine Funktion, die den Standards von Cygnus nicht mehr entspricht
    * setSite, setUsername, setPassword sind obsolet
    
    Sonstiges
    * Rückgabewerte von serialisierten Arrays auf direkt weiter verarbeitbare Arrays umgestellt (Ausnahme movePage)
    * Funktionen, die entweder einen bestimmten Wert (String, Array, Integer) oder den boolschen Wert false zurückgeben stattdessen auf null umgestellt, da dieser Wert intuitiver ist und die Kompatibilität mit PHP 7.1 erhöht
    * Kleinere Korrekturen
    
    Ref T823, T841, T872, T875
    
    Reviewers: Luke, O1 Luke081515Bot, Morten_Haan
    
    Reviewed By: Luke, O1 Luke081515Bot, Morten_Haan
    
    Subscribers: Luke, Morten_Haan
    
    Tags: #cygnus
    
    Maniphest Tasks: T875, T872, T841, T823
    
    Differential Revision: https://rcm-2.wmflabs.org/D422
    1f070179
settings.example.json 441 Bytes