-
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