[drgrice1]

This is something I have been intending to do since the conversion to Mojolicious. There is no need for a pre_header_initialize and initialize method anymore. Just one is needed. Furthermore, the name pre_header_initialize no longer has the meaning it was named for. That being an initialization before the header. The header is no longer set like it was with modperl and this method is not before that.

Technically the initialize method is removed and the pre_header_initialize method renamed to the initialize method in terms of when the method is called from the WeBWorK::ContentGenerator::go method. The only real difference between the methods is that the reply_with_file or reply_with_redirect methods had to be called from the pre_header_initialize method, and would not work from the initialize method.

For most WeBWorK::ContentGenerator subclasses all that was needed was to rename the pre_header_initialize method to initialize. However, there were a few that called both methods, and so the contents of the two methods had to be carefully merged to preserve calling order and properly handle early returns.

As to the header method, it has done nothing since the conversion to Mojolicious. Its return value is entirely ignored and nothing overrides it.