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.

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

[REF] Cleanup on permission code #19796

Merged
merged 1 commit into from
Mar 14, 2021
Merged

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Mar 12, 2021

Overview

[REF] Cleanup on permission code

Before

Ug

After

Soothing calm

Technical Details

The function returns either an array of arrays or an array of the
first key in the array, depending on the 'desriptions' parameter.

This cleans it up so that instead of handling the descriptions
parameter in multiple places we get the array
in all cases and do the formatting at the end

Comments

The function returns either an array of arrays or an array of the
first key in the array, depending on the 'desriptions' parameter.

This cleans it up so that instead of handling the descriptions
parameter in multiple places we get the array
in all cases and do the formatting at the end
@civibot
Copy link

civibot bot commented Mar 12, 2021

(Standard links)

@eileenmcnaughton
Copy link
Contributor Author

@colemanw can you merge this so I can simplify the super-duper PR? (Should help people looking at it / discussing)

@colemanw
Copy link
Member

Yes. Refactor looks sensible & tests pass.

@colemanw colemanw merged commit d600909 into civicrm:master Mar 14, 2021
@colemanw colemanw deleted the desc branch March 14, 2021 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants