WordPress.org

Make WordPress Core

Opened 7 months ago

Last modified 2 months ago

#41221 assigned defect (bug)

Throw error notice after wp-header-end

Reported by: shivapoudel Owned by: swissspidy
Milestone: 5.0 Priority: normal
Severity: normal Version: 4.8
Component: Upgrade/Install Keywords: shiny-updates has-patch
Focuses: ui, javascript, administration Cc:

Description

Append admin notice for AJAX updates after wp-header-end

Attachments (6)

41221.diff (756 bytes) - added by shivapoudel 7 months ago.
Display admin AJAX error notice after wp-header-end
Capture.PNG (398.7 KB) - added by shivapoudel 7 months ago.
Error Image
Screenshot-1.png (31.2 KB) - added by shivapoudel 3 months ago.
Browse Plugins > Recently active section
Screenshot-2.png (31.2 KB) - added by shivapoudel 3 months ago.
Select Delete in Bulk action without any selection there
Screenshot-3.png (41.2 KB) - added by shivapoudel 3 months ago.
After clicking submit you will be throw error which this Patch resolves :)
Screenshot-4.png (34.6 KB) - added by shivapoudel 3 months ago.
After implementation of the patch it will look like this :)

Download all attachments as: .zip

Change History (11)

@shivapoudel
7 months ago

Display admin AJAX error notice after wp-header-end

@shivapoudel
7 months ago

Error Image

#1 @swissspidy
7 months ago

  • Component changed from Administration to Upgrade/Install
  • Keywords shiny-updates has-patch added

#2 @shivapoudel
3 months ago

@swissspidy Will this be patched in upcoming WordPress 4.9 version or not, Please do reply :)

#3 @swissspidy
3 months ago

  • Keywords needs-screenshots added

It's a bit late for 4.9.

Can you explain a bit on why this change is needed and how things look like without the patch?

@shivapoudel
3 months ago

Browse Plugins > Recently active section

@shivapoudel
3 months ago

Select Delete in Bulk action without any selection there

@shivapoudel
3 months ago

After clicking submit you will be throw error which this Patch resolves :)

#4 @shivapoudel
3 months ago

  • Keywords needs-screenshots removed

Hey @swissspidy I was missing the follow up emails for this ticket or was located somewhere in my inbox. Anyway sorry for the belated reply :)

Since reporting this ticket I have some modification hardly but today I will illustrate the simply way and what this patch does.

First I will surf my Development environment Plugins page > Recently active section.
Note: Since I have only WooCommerce in this section I will prefer this for steps to be clear :)

Without any selection I will choose Delete in bulk actions and will submit for AJAX process.
And it will throw the error message regarding no selection for delete action. AND this patch is especially for this step.

AFAIK this bug was after wp-header-end implementation maybe on 4.8 and punting this patch in 4.9 will make sense. This will not be any life breaking change IMO.

Thanks and sorry for the belated reply!

@shivapoudel
3 months ago

After implementation of the patch it will look like this :)

#5 @swissspidy
2 months ago

  • Milestone changed from Awaiting Review to 5.0
  • Owner set to swissspidy
  • Status changed from new to assigned

Got it, thanks!

Note: See TracTickets for help on using tickets.