Hallo Pyfisch,

ich finde es großartig, wie Du Dich beim pwb als Coder einbringst und es ist immer doof, beim Review ein -1 oder -2 zu vergeben, weil es immer den Eindruck hinterläßt, daß Deine Arbeit nicht geschätzt wird. Sei versichert, ic schätze sie sehr und da CR ist rein technischer Natur, das der Verbesserung dient. Das mal so vorweg.

Jetzt zu einigen Punkten: Deine Änderungen in category_redirekt fand ich sehr elegant, weshalb ich sie auch in das Framework übernommen habe. Leider hat dies in bestimmte Fällen zu Fehlern geführt [1], was ich auch übersehen habe. Daher habe ich Deine Änderungen wieder zurücknehmen müssen [2]. Vielleicht findest Du noch einen Weg, der die lokale Methode weitgehend überflüssig macht ohne die fehlerhafte Log-Funktion.

category.py sollte erst dann ein Rewrite bekommen, sobald alle Funktionen aus der compat-Branch übernommen worden sind. Sonst wird das später umso schwieriger. Deshalb hier erstmal ein CR-2. Außerdem solltest Du die Änderungen mehr dokumentieren (Siebrand kommentierte das auch). Sonst ist der Review enorm schwierig, weil man sich in das Programm reindenken muß. Du hast da auch einen Breaking change beim Construktor. Bitte noch überprüfen, ob der Bot nicht anderweitig benutzt wird, ggf. mit dem @deprecate_arg decorator umbiegen.

LG und guten Rutsch morgen

xqt

[1] https://bugzilla.wikimedia.org/process_bug.cgi [2] https://gerrit.wikimedia.org/r/#/c/104505/