Re: [Wikitech-l] Splitting up EditPage::showEditForm()

Hi, The method showEditForm(), in the class EditPage, takes care of displaying nearly the entire edit page. I would like to move the last section of this method, which handles displaying the bottom of the edit page (everything below the edit box) into its own method, perhaps called showFooter(). (This basically corresponds to lines 1506 to 1566 of the current EditPage.php.) showEditForm() would then call that new method. The reason for this move is so that the extension Semantic Forms can subclass EditPage and override the new method. But you could also make the case that it would improve readability, modularity, whatever else. Any thoughts/objections?