It's great that Drupal's default Content (at admin/content) has been replaced with a more-flexible View. How about doing the same for the People page (at admin/people)? That would let admins:

  1. Customize the interface
  2. Apply more bulk operations to people
  3. Apply bulk operations to all people, not just on a screen-by-screen basis

...and so forth. It would also make this screen more consistent with others in CK, and it's a fairly easy fix. Thanks!

CommentFileSizeAuthor
#5 convert_people_screen-1871678-5.patch2.73 KBmglaman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tgeller’s picture

Version: 7.x-2.0 » 7.x-2.x-dev
lsolesen’s picture

Issue summary: View changes

Would admin views be an idea instead for both

bojanz’s picture

admin_views ships its own Content view, which would conflict with commerce_backoffice. That's why we didn't add it in the first place.
Perhaps we could add it, and then disable its content view.

mglaman’s picture

Issue tags: +sprint

I could see this being viable, especially since most sites use it. If we're also implementing a fix to disable the admin_views content override, then we'll mitigate that bug for other users, too, who might experience it if we don't add admin_views.

Thoughts, bojanz?

mglaman’s picture

Status: Active » Needs review
FileSize
2.73 KB

Turns out Commerce Backoffice does disable admin_views conflicts!

/**
 * Implements hook_views_default_views_alter().
 *
 * Disable the content view provided by the admin_views module.
 */
function commerce_backoffice_content_views_default_views_alter(&$views) {
  if (isset($views['admin_views_node'])) {
    $views['admin_views_node']->disabled = TRUE;
  }
  if (isset($views['admin_views_comment'])) {
    $views['admin_views_comment']->disabled = TRUE;
  }
}

Build: https://travis-ci.org/commerceguys/commerce_kickstart/builds/71805324 (with tests checking its working)

  • mglaman committed 2477766 on 7.x-2.x
    Merge pull request #136 from mglaman/1871678-add-admin-views
    
    Issue #...
  • mglaman committed deab3f4 on 7.x-2.x
    Issue #1871678: Convert People screen to use Views (like the Content...
mglaman’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.