Make WordPress Core

Opened 11 years ago

Closed 7 years ago

Last modified 7 years ago

#10480 closed enhancement (maybelater)

Need method to display custom errors for quickedit

Reported by: sirzooro Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.8.2
Component: Quick/Bulk Edit Keywords:
Focuses: Cc:


Some time ago I wrote Wypiekacz plugin. It checks if post satisfies some rules. When check result is negative, plugin displays message for the logged user. Unfortunately when post is edited with quickedit, user do not get any error message. It will be good do add mechanism do return custom error message to the client for quickedit.

Attachments (1)

edit_result.diff (1.5 KB) - added by sirzooro 11 years ago.
1st attempt

Download all attachments as: .zip

Change History (8)

11 years ago

1st attempt

#1 @sirzooro
11 years ago

  • Keywords has-patch needs-testing 2nd-opinion added; needs-patch removed

#2 @ryan
10 years ago

  • Milestone changed from 2.9 to 3.0

#3 @Denis-de-Bernardy
10 years ago

  • Keywords reporter-feedback added; 2nd-opinion removed

shouldn't there also be an is_wp_error() check over in wp_insert_post()?

#4 @nacin
10 years ago

  • Component changed from General to Quick Edit
  • Keywords has-patch needs-testing removed
  • Milestone changed from 3.0 to Future Release

#5 @sirzooro
9 years ago

I will answer Denis' question later.

I have found another case when this update is useful - do not create/update post if custom validation done in wp_insert_post_data fails - no matter if editing is done via QuickEdit or normal editor. Now the only way is to hook the query filter and replace SQL with some invalid one.

Related: #18266

#6 @iseulde
7 years ago

  • Resolution set to invalid
  • Status changed from new to closed

No feedback in over two years.

#7 @helen
7 years ago

  • Keywords reporter-feedback removed
  • Milestone Future Release deleted
  • Resolution changed from invalid to maybelater

Quick/Bulk Edit seem like they could use a lot of love in general. Setting to maybelater - if/when those get attention, this would be worth circling back to.

Note: See TracTickets for help on using tickets.