Make WordPress Core

Opened 5 years ago

Closed 8 months ago

Last modified 8 months 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 5 years ago.
1st attempt

Download all attachments as: .zip

Change History (8)

sirzooro5 years ago

1st attempt

comment:1 sirzooro5 years ago

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

comment:2 ryan4 years ago

  • Milestone changed from 2.9 to 3.0

comment:3 Denis-de-Bernardy4 years ago

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

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

comment:4 nacin4 years ago

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

comment:5 sirzooro3 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

comment:6 avryl8 months ago

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

No feedback in over two years.

comment:7 helen8 months 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.