Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mass Assign More Than 1 Page Doesn't Complete #9476

Closed
cameronblaikie opened this issue Jan 28, 2022 · 2 comments · Fixed by #9528
Closed

Mass Assign More Than 1 Page Doesn't Complete #9476

cameronblaikie opened this issue Jan 28, 2022 · 2 comments · Fixed by #9528
Labels
Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Type: Bug Bugs within the core SuiteCRM codebase

Comments

@cameronblaikie
Copy link
Contributor

Issue

When you set your page size to 20 and you have say 100 records, and you select 40 of them and hit mass assign a security group and try set them only the last page will update the rest before will be ignored and not updated.

Expected Behavior

all records should be updated instead of just the last 20 you selected

Actual Behavior

Possible Fix

Steps to Reproduce

  1. See Issue ^^

Context

Your Environment

  • SuiteCRM Version used:
  • Browser name and version (e.g. Chrome Version 51.0.2704.63 (64-bit)):
  • Environment name and version (e.g. MySQL, PHP 7):
  • Operating System and version (e.g Ubuntu 16.04):
@Mac-Rae
Copy link
Contributor

Mac-Rae commented Feb 2, 2022

Hi @cameronblaikie,

Seems to be fine on the latest 7.12, could you toss me more information regarding your environment?

@Mac-Rae Mac-Rae added the Status:Requires Updates Issues & PRs which requires input or update from the author label Feb 2, 2022
@johnM2401 johnM2401 added Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Type: Bug Bugs within the core SuiteCRM codebase and removed Status:Requires Updates Issues & PRs which requires input or update from the author labels Feb 10, 2022
JackBuchanan added a commit to JackBuchanan/SuiteCRM that referenced this issue Feb 28, 2022
@JackBuchanan
Fix salesagility#9476 - Mass assign security groups only assigns sele…
4d1cc52 
…cted on current page

Change how UID array is built in assignGroups
@JackBuchanan JackBuchanan mentioned this issue Feb 28, 2022
Merged
6 tasks
@JackBuchanan
Copy link
Contributor

Just added a fix, I doubled checked to see if this was still an issue and I can confirm the issue exists in 7.12.4

jack7anderson7 pushed a commit to JackBuchanan/SuiteCRM that referenced this issue Jul 27, 2022
@JackBuchanan @jack7anderson7
Fix salesagility#9476 - Mass assign security groups only assigns sele…
90dff98 
…cted on current page

Change how UID array is built in assignGroups
jack7anderson7 pushed a commit that referenced this issue Apr 14, 2023
@JackBuchanan @jack7anderson7
Fix #9476 - Mass assign security groups only assigns selected on curr…
ef23047 
…ent page

Change how UID array is built in assignGroups
jack7anderson7 pushed a commit that referenced this issue Apr 24, 2023
@JackBuchanan @jack7anderson7
Fix #9476 - Mass assign security groups only assigns selected on curr…
56ffd21 
…ent page

Change how UID array is built in assignGroups
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Type: Bug Bugs within the core SuiteCRM codebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #9476 - Mass assign security groups only assigns selected on current page JackBuchanan/SuiteCRM
4 participants
@johnM2401 @JackBuchanan @cameronblaikie @Mac-Rae