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

Print report or export report if user have access report permission #23616

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

pradpnayak
Copy link
Contributor

Overview

https://lab.civicrm.org/dev/core/-/issues/2540

Before

when disabling the “access Report Criteria” permission, the "Print report" and "Print to PDF" functions on a report appear, but do not function.
After

when disabling the “access Report Criteria” permission, the "Print report" and "Print to PDF" functions on a report appear to work

Technical Details

When “access Report Criteria” is not set to user role, $this->_params['fields'] is empty hence $this->_noFields is not set causing $this->_params to get over-ridden with formvalues stored in the database, hence task parameter is lost is $this->_params which is used to set report mode in processReportMode().

@civibot
Copy link

civibot bot commented May 28, 2022

(Standard links)

@civibot civibot bot added the master label May 28, 2022
@pradpnayak
Copy link
Contributor Author

An alternate PR for this fix is #23617

@adevapp
Copy link

adevapp commented May 29, 2022

Tested and works.

@eileenmcnaughton
Copy link
Contributor

@pradpnayak @yashodha do either of you have a preference between the 2 - I see @adam-devapp has tested this which puts it ahead :-)

@eileenmcnaughton eileenmcnaughton added the merge ready PR will be merged after a few days if there are no objections label May 31, 2022
@eileenmcnaughton
Copy link
Contributor

I guess since this has been UI tested & the other has ?? that's a vote for this one .... @mlutfy I have some memory of you engaging in something related to this in the past?

@eileenmcnaughton eileenmcnaughton merged commit 6fa097c into civicrm:master Jun 2, 2022
@pradpnayak pradpnayak deleted the dev2540-1 branch June 5, 2022 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master merge ready PR will be merged after a few days if there are no objections
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants