md5 wrote:

After some discussion on English Wiktionary's Beer Parlour, it may make more sense for this to apply to more than just new sections. If that were the case, it would probably make more sense for it to be called "preloadsummary" instead of "preloadtitle". The patch would be more complicated for that case since the interaction with the preloaded section comments would need to be worked out in the case of editing an existing section.

conrad.irwin wrote:

I assumed that preload itself doesn't work for editing already existing sections, wouldn't that overwrite the contents that should be there? I think that the "preloadtitle" is an adequate name for the feature - as that is what it would be used for. This would be very useful for creating RfV/RfD (add to page) links as well.

ran.arigur wrote:

I assumed that preload itself doesn't work for editing already existing
sections [...]

Correct. preloadsummary, however, might be useful more generally; it could
perhaps be appended to the default summary. (I can't say that I care much one
way or the other. This simply came up because my temporary JavaScript solution
doesn't check to make sure it's on a section=new page, so would support
preloadtitle on any page. If people don't like that, we can stick with
preloadtitle.)

md5 wrote:

Has anyone had a chance to look at this patch? I'd be happy to make any changes necessary to get it into SVN.

Patch applied as posted in r31329

ran.arigur wrote:

Thanks! :-)

md5 wrote:

Replacement for previous patch...

The previous patch just went live and did not work correctly. The value of $this->summary was being wiped out after being set (in the initialiseForm() method).

This revised patch reverses the earlier patch and does the check for "preloadtitle" inside of initialiseForm() instead of inside importFormData(). It adds a global import of $wgRequest in initialiseForm() because the request parameters are not available in that method otherwise.

I have tested this patch in a local MediaWiki installation, unlike the previous patch (*hanging head in shame*).

Attached:

I fixed this issue in r31419. The line in initialiseForm() blanking $this->summary was completely useless as $this->summary is blank by default anyway.

alexsm333 wrote:

Would it be possible to make "preloadtitle" work on section=0?

That's another case when summary is blank, so it would be nice to have the option to pre-populate it from URL.