Quick edit issues in Opera

[mrmist]

Seems to be an issue with Quickedit publish / sticky post.

1. The "sticky" button has no effect on pending posts (this may be intended, but then the button should be ghosted until valid.)

2. When a Pending post is atop the edit all posts list, hitting Quick Edit, change from pending to Published, then save sometimes causes the post to vanish from the list, without changing status. This seems to more reliably be a bug with posts that have only recently been submitted for review, so it may be time-dependant. (Bug noted in Opera.) Also note that when this bug stops being a bug, bug 1 also stops.

[mrmist]

These appear to be Opera (9.51) issues, as I can reproduce them whilst at the same time checking in IE where they do not occur.

It is not limited to changes to published status and sticky, it affects all the quick edit functionality for "newish" posts.

[mrmist]

Also reported in Opera 9.61

[azaozz]

When this happens, are there any js errors reported (Tools, Advanced, Error Console, set to show only js errors).

[DD32]

I cant reproduce the row disapearing (I reported it in #8137 which is what ​http://trac.wordpress.org/ticket/8115#comment:3 comment 3 was refering to) after the last few QE commits, But i could reproduce it *every* time yesterday.

The sticky option seems to be sticking too for me right now.

[mrmist]

Agreed both of these are working now as of 9611.

The whole area seems to have changed in design, so I guess one of the intervening changes has sorted it.

[DD32]

Problem has surfaced again today running r9655

No Javascript errors are appearing either.

Cancel works correctly, But clicking Save results in the table row never re-appearing.

[azaozz]

Still can't reproduce this. It seems to resurface every time inline-edit-tax.js is updated. Perhaps Opera hangs on to js files for longer/checks for updates less frequently for them.

Also jQuery suppresses some js errors. For example jQuery('div#mydiv').doSomething won't produce an error if there is no DIV with id="mydiv" on the page but "normal" js would show a type error that "mydiv" is null.

Perhaps adding some debug statements in inline-edit-tax.js would bring more info about what causes this.

[mrmist]

Not seeing the problem at the moment in Opera.

[DD32]

Next time it happens i'll scatter debugging stuff through the files.. Even though i cleared the cache, and even loaded it under a different profile, I couldnt get it to work. Now that i've sat down and gone to debug it, its working perfectly.

[DD32]

It appears that the row disapearing (for me at least) was caused by the returned HTML most likely being non-valid, due to PHP Notice's.

attachment 8115.diff checks that the post_parent field is set before using it, Passes the seconds through quick edit (But doesnt show it as a field in QE), and checks that 'keep_private' is in the post data before using it.

[azaozz]

(In [9712]) QE: fix notices and check if the returned string contains the row HTML, props DD32, fixes #8115