Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-7415

Joomla.php should us JError rather than Error.php

          Details

          • Type: Bug
          • Status: Done/Fixed
          • Priority: Minor
          • Resolution: Fixed/Completed
          • Affects Version/s: 3.3.2
          • Fix Version/s: 3.4.alpha
          • Component/s: Core CiviCRM
          • Labels:
            None

            Description

            In CRM/Utils/System/Joomla.php static function permissionDenied( ) { which is around line 338

            CRM_Core_Error::fatal( ts( 'You do not have permission to access this page' ) is used.

            It would be preferable to use the native Joomla function which will give the same information but formatted in a manner consistent with the design if the Joomla site and in the form that Joomla developers expect.

            Proposed code to replace what is there:
            JError::raiseError(403, ALERTNOTAUTH );

            Since this is a CMS specific file anyway this shouldn't create a problem, and in fact solves some UX issues.

              Attachments

                Activity

                  People

                  • Assignee:
                    lobo Donald A. Lobo
                    Reporter:
                    mcsmom Elin Waring
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: