We should remove ProfileViewsData, it is only used to override core's bulk form handler.
However, our bulk form handler extends from a deprecated class, and is only used to customize the empty selection text.

I'm fine with using the default core text ("No items selected") until core makes the text configurable in #2909423: Get rid of BulkForm subclasses when they just override emptySelectedMessage.

CommentFileSizeAuthor
#3 3044063-3.patch2.1 KBmglaman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz created an issue. See original summary.

mglaman’s picture

Assigned: Unassigned » mglaman

Works for me. Kills the amount of code to maintain.

mglaman’s picture

Status: Active » Needs review
FileSize
2.1 KB

Patch

  • mglaman committed 0f5dc7b on 8.x-1.x
    Issue #3044063 by mglaman: Remove the custom bulk form implementation
    
mglaman’s picture

Assigned: mglaman » Unassigned
Status: Needs review » Fixed

Fixed

Status: Fixed » Closed (fixed)

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