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

Homescreen may fail to load after deleting a report #10410

Closed
simonr44 opened this issue Apr 22, 2024 · 2 comments · Fixed by #10411
Closed

Homescreen may fail to load after deleting a report #10410

simonr44 opened this issue Apr 22, 2024 · 2 comments · Fixed by #10411
Labels
Area: Dashlets Issues & PRs related to all things regarding the dashlets Area: Reports Issues & PRs related to all things regarding reports Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Severity: Major Significant impact/severe disruption Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Type: Bug Bugs within the core SuiteCRM codebase Type:Duplicate Issues & PRs which are a duplicate of an existing issue/PR

Comments

@simonr44
Copy link
Contributor

Issue

The reports dashlet allows you to add reports to your homescreen.
If you delete a report which is used in a reports dashlet, the home screen fails to load and throws a 500 error.

Expected Behavior

The dashboard should continue to load, even after deleting a report which is used by the reports dashlet.

Actual Behavior

Uncaught Error: Attempt to assign property "user_parameters" on bool in /var/www/modules/AOR_Reports/Dashlets/AORReportsDashlet/AORReportsDashlet.php:44

Possible Fix

To follow

Steps to Reproduce

  1. Create a test report
  2. Add a reports dashlet to your dashboard
  3. Assign the test report to your new reports dashlet
  4. Confirm the home screen loads
  5. Delete the report
  6. Confirm the home screen fails to load, with the error shown above in the logs

Context

When this happens, the main workarounds are to clear user preferences or restore the deleted report.
This fix prevents the unrecoverable error and allows the user to interact with the dashlet as normal.

Your Environment

  • SuiteCRM Version used: 7.14.3
  • Browser name and version (e.g. Chrome Version 51.0.2704.63 (64-bit)): Chrome
  • Environment name and version (e.g. MySQL, PHP 7): PHP8.2, MariaDB
  • Operating System and version (e.g Ubuntu 16.04): Debian
simonr44 added a commit to simonr44/SuiteCRM that referenced this issue Apr 22, 2024
@simonr44 simonr44 linked a pull request Apr 22, 2024 that will close this issue
6 tasks
@serhiisamko091184 serhiisamko091184 added the Status:Requires Updates Issues & PRs which requires input or update from the author label Apr 23, 2024
@serhiisamko091184
Copy link
Contributor

Hello @simonr44,

thanks for raising the issue.

This appears to be a duplicate of: #10353
Please let us know if you believe otherwise.

Regards,
Serhii

@johnM2401 johnM2401 added Type: Bug Bugs within the core SuiteCRM codebase Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Type:Duplicate Issues & PRs which are a duplicate of an existing issue/PR Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Area: Reports Issues & PRs related to all things regarding reports Area: Dashlets Issues & PRs related to all things regarding the dashlets Severity: Major Significant impact/severe disruption labels Apr 23, 2024
@simonr44
Copy link
Contributor Author

Apologies I didn't spot the duplicate. Not sure if you have a preference on how to resolve the issue as we have taken different approaches here.

@serhiisamko091184 serhiisamko091184 removed the Status:Requires Updates Issues & PRs which requires input or update from the author label May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Dashlets Issues & PRs related to all things regarding the dashlets Area: Reports Issues & PRs related to all things regarding reports Priority:Important Issues & PRs that are important; broken functions, errors - there are workarounds Severity: Major Significant impact/severe disruption Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Type: Bug Bugs within the core SuiteCRM codebase Type:Duplicate Issues & PRs which are a duplicate of an existing issue/PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants