WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 3 months ago

#16793 reopened defect (bug)

Bulk inline post edit doesn't show categories or tags if the CPT doesn't support titles

Reported by: scribu Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Quick/Bulk Edit Keywords: has-patch
Focuses: Cc:

Description

  1. Register a CPT that doesn't support titles:
add_action( 'init', function() {
	register_post_type( 'foo', array(
		'show_ui' => true,
		'label' => 'Foo',
		'supports' => array( 'editor' ),
		'taxonomies' => array( 'category', 'post_tag' )
	) );
} );
  1. Create a 'foo' post.
  1. Go to wp-admin/edit.php?post_type=foo and choose the 'Edit' bulk action.

Expected behaviour: shows the category checkbox list and the tag textarea.

Actual behaviour: only the status dropdown is shown.

Attachments (3)

16793.diff (524 bytes) - added by scribu 3 years ago.
16793.2.diff (1.0 KB) - added by scribu 3 years ago.
optimize selectors
16793.3.diff (2.3 KB) - added by scribu 3 years ago.
selector cleanup + DRY

Download all attachments as: .zip

Change History (13)

scribu3 years ago

comment:1 markmcwilliams3 years ago

I found it worked before, and after Scribu; unless I'm holding it wrong?

comment:2 scribu3 years ago

If you're using Chrome, without the patch, you should also be getting a JS error in the console:

Uncaught TypeError: Cannot read property 'firstChild' of null

comment:3 scribu3 years ago

Note that we're talking about the Edit option in the dropdown, not the quick action Edit link on each post.

comment:4 markmcwilliams3 years ago

OK so I was holding it wrong, I can confirm the initial error, and that the patch works! :)

scribu3 years ago

optimize selectors

comment:5 ryan3 years ago

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

(In [17504]) Don't perform taxonomy queries for single post requests. Props scribu. fixes #16793 for trunk

scribu3 years ago

selector cleanup + DRY

comment:6 scribu3 years ago

16793.3.diff:

Contrary to popular belief, $('some selector', context) is not faster than context.find('some selector'). It's just harder to read.

Also, there was a callback written twice, once for qeRow.keyup and again for bulkRow.keyup.

Last edited 3 years ago by scribu (previous) (diff)

comment:7 scribu3 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:8 somatic3 years ago

  • Cc israel@… added

another casualty of this problem is that any javascript (and jQuery) that would be executed on the wp-admin/edit.php?post_type=foo screen can't run, as the javascript engine stops at that initial Uncaught TypeError: Cannot read property 'firstChild' of null error. I had been adding some additional sorting options via jQuery, and couldn't figure out why they appeared on some post type edit lists, but not this one, until I found this ticket...

I can confirm that add the supports => titles does fix the problem, but it's not absolute: I have two CPTS, registered with basically identical args, and neither supports titles - but one manifests the javascript error on edit.php and the other does not...

Version 0, edited 3 years ago by somatic (next)

comment:9 gizburdt6 months ago

The expected behaviour is not showing up for me

comment:10 ocean903 months ago

  • Component changed from Administration to Quick/Bulk Edit
Note: See TracTickets for help on using tickets.