[12:27:33] (CR) Joeytje50: "I was thinking about re-doing this change, but using the API, to also prevent edit-conflicts when rapidly changing values on the same page" [extensions/PageForms] - https://gerrit.wikimedia.org/r/352771 (owner: Joeytje50) [14:01:26] (CR) Yaron Koren: "Hi - the file /includes/PF_FormPrinter.php handles most of the display and running of a form, mostly in the large function formHTML()." [extensions/PageForms] - https://gerrit.wikimedia.org/r/352771 (owner: Joeytje50) [21:26:31] (CR) Joeytje50: "Oh wow, that function looks really huge... Almost 1000 lines of code... Wouldn't it be better to split up that function to at least 2 func" [extensions/PageForms] - https://gerrit.wikimedia.org/r/352771 (owner: Joeytje50)