Problem/Motivation

After submitting the bulk edit form there is no Status Message to indicate that the edit is successful.

Steps to reproduce

1. Bulk Edit Contents
2. Submit the form
3. Notice that there is no status message to indicate that the edit is successful

Proposed resolution

Add on form submit status message to notify users that edit is successful

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

erwindaez created an issue. See original summary.

anybody’s picture

Version: 8.x-2.9 » 8.x-2.x-dev
Issue tags: +Novice

Yeah that makes sense, showing the results and checking the status. Could someone please prepare a proper MR?

sonyavpaa made their first commit to this issue’s fork.

jagraj_singh_gill’s picture

Assigned: Unassigned » jagraj_singh_gill

I am working on this request.

jagraj_singh_gill’s picture

Assigned: jagraj_singh_gill » Unassigned
Status: Active » Needs review

I have created the MR for this. I have applied the success/failure messages by checking the number of successful and failed entities. And also applied the logger for failed entities. Please review once.
Thank You !!

papaaldrin’s picture

StatusFileSize
new41.9 KB

It's working properly on my end and here is the screenshot. (This is my first contribution.)

Only local images are allowed.

papaaldrin’s picture

Unfortunately, I see that the PHPUnit is failing. I will look into it to make it work.

sonyavpaa’s picture

Created a bit shorter version of the original MR, adds a status message after editing

roberttabigue’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new197.46 KB

Hi,

I installed and enabled the Views Bulk Edit module against 8.x-2.x-dev on my D10 site, and confirmed the Status message is now working after applying MR!33.

Please see the attached file for reference.

I'm moving this now to ‘RTBC’.

Thank you!

anybody’s picture

Status: Reviewed & tested by the community » Needs work

Thanks. I think we should improve the wording a bit. Any native EN speaker here?

I'd vote for something like

Successfully modified %total_count entities

. Any similar string in core?

chhavi.sharma’s picture

Assigned: Unassigned » chhavi.sharma

Working on it.

chhavi.sharma’s picture

Assigned: chhavi.sharma » Unassigned
Status: Needs work » Needs review

Modifed the status message as per suggestion.

chhavi.sharma’s picture

StatusFileSize
new25.86 KB

Attaching the screenshot for reference.
Only local images are allowed.

anybody’s picture

Status: Needs review » Needs work
Issue tags: +Needs tests

Thank you! Sadly both MR's still have failing tests and we should add tests for the new message to appear, please.
Code style is also incorrect.

chhavi.sharma’s picture

Assigned: Unassigned » chhavi.sharma

Working on the code style and trying to make pipeline green.

chhavi.sharma’s picture

Assigned: chhavi.sharma » Unassigned

Hi @anybody,
Fixed all the phpcs issues and made phpcs pass. But still there is phpunit failing. I have no prior experience of writing tests. Therefore, unassigning myself from the issue.

voj changed the visibility of the branch 3470852-status-message-2 to hidden.

voj changed the visibility of the branch 3470852-status-message-2 to active.

n-m-daz’s picture

Issue tags: +Singapore2024

I am currently working on this at DrupalCon SG 2024 until the end of the day.

usingsession made their first commit to this issue’s fork.

usingsession’s picture

Status: Needs work » Needs review

I fixed the test, but I made a mistake with the branch name (Sorry about that).

anybody’s picture

Status: Needs review » Reviewed & tested by the community

Great, thank you!

anybody’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs tests
anybody’s picture

Version: 8.x-2.x-dev » 3.x-dev
chhavi.sharma’s picture

Hi @anybody,
I noticed that a new MR was generated for this issue which includes the changes I did in MR 33 previously. While I appreciate the progress that the issue has been fixed, I’d like to kindly request due credit for my contribution to this issue as it motivates me to continue contributing to Drupal.
Looking forward to your response!

anybody’s picture

@chhavi.sharma yes of course, sorry!

Status: Fixed » Closed (fixed)

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

rajab natshah’s picture

Priority: Normal » Major
Issue tags: +PHP 8.4

Thank you for this fix.
Hoping for a release.

rajab natshah’s picture

Hopping for a tag release.
Thank you :)

rajab natshah’s picture

Priority: Major » Critical
anybody’s picture

Priority: Critical » Major

Not really critical, I think... @rajab natshah did you test the current dev branch? Everything else fine? Then I'd be fine to tag a new release of course. Thanks for your help.

rajab natshah’s picture

anybody’s picture

@rajab natshah what about #3563973: Deprecated nullable parameter? Sorry very busy currently.

rajab natshah’s picture

Thank you for your time Julian.

  • Both created a new issue.
  • then created an issue fork
  • But noticed that it was already fixed in the DEV branch.

So we only managed with a patch

Created: 7 Dec 2025 : #3561886: Implicit nullable AccountInterface parameters for PHP 8.4 compatibility in Views Bulk Edit ( I closed mine as wont fix as it was already fixed in this issue - managing with local patches, and a patch from the commit)
Created: 18 Dec 2025 : #3563973: Deprecated nullable parameter

anybody’s picture

@rajab natshah so TL;DR: 3.x-dev is fine, we just need a tagged release, right?

rajab natshah’s picture

Yes for sure, the dev was fine.
I had 5 testing rounds before the release
and
2 post release testing rounds ( ~11.3.0, and ~10.6.0 )

Thanks for the release, Julian, and your time today.

anybody’s picture

Thank YOU @rajab natshah it's always a pleasure! :)