When restoring a Page revision, a blank success message is shown

[Viper007Bond]

If you restore a Page revision, the yellow success box at the top is empty. This is because the message is missing.

[Viper007Bond]

Suggest it be backported to 2.6.1.

[azaozz]

(In [8506]) Add "Page restored" message. Props Viper007Bond, fixes #7437 for 2.7

[azaozz]

Re-open for 2.6.1

[azaozz]

(In [8507]) Add "Page restored" message. Props Viper007Bond, fixes #7437 for 2.6.1

[DD32]

Just a quick thought: just using $_GETrevision? could cause a undefined index warning, also means its calling for the revision title regardless.. It could be added to an if(isset($_GETrevision?)) or similar?

[Viper007Bond]

DD32: I just copy/pasted from the post version, but yeah it would cause an undefined index if you messed with the URL. However if you use a clean install of ?WordPress, it should be impossible to get such an error as WP always passes it.

Not sure if it's worth the trouble or not to protect against a warning caused by a manual URL creation.

[DD32]

if you messed with the URL.

My thought is that that page is loaded on urls that do not include ?revision=123 etc, which would cause it.

I've seen notice level errors poping up on some dev severs where by default its set to show notice errors, AFAIK some effort is made to avoid non-set array items such as notices judging by the other tickets which get commited to remove them.

[Viper007Bond]

Replying to DD32:

My thought is that that page is loaded on urls that do not include ?revision=123 etc, which would cause it.

Oh, right, that wp_post_revision_title() gets called on every single page load. We should wrap the whole thing in a $_GET['message'] conditional.

[azaozz]

(In [8508]) Stop PHP warnings when $_GET[revision] is not set. Props DD32, see #7437

[azaozz]

(In [8509]) Stop PHP warnings when $_GET[revision] is not set. Props DD32, see #7437, for 2.6

[Viper007Bond]

Not that it really matters, but if $_GET['revision'] is not set is not set but $_GET['message'] == 5, then a blank yellow box will be shown again. :P

[azaozz]

Replying to Viper007Bond:

Not that it really matters, but if $_GET['revision'] is not set is not set but $_GET['message'] == 5, then a blank yellow box will be shown again. :P

Yes, however message[5] is only shown after restoring revision, and $_GET[revision] is always set then, no way to restore it otherwise.