CRM-8630 Event Tabs Broken without Drupal Clean URLs Enabled

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Major
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.0.5
    • Fix Version/s: 3.4.6
    • Component/s: None
    • Labels:
      None

      Description

      As per the issue first documented in this forum post: http://forum.civicrm.org/index.php?topic=20907.0

      I am unable to switch tabs and then save the changes on that tab as the form post URL gets mangled - I have seen this happen on Event configuration, Event Templates tabs. The workaround, for Events at least, is to go from the Manage Events page, and click to edit the direct tab you want, then Save and return to the page and repeat for other tabs.

      I can replicate this issue in v4.0.5. I am unable to enable clean URLs on my drupal installation. This effectively breaks all of the Event functionality in my setup.

      The issue is in: templates/CRM/common/TabHeader.tpl (Line 65-67 in v4.0.5) where the URL gets split by the ? character. In the case of my site (http://192.168.79.21/drupal/index.php?q=civicrm/event/manage/registration&action=update&reset=1&id=7) this means that the post URL becomes: http://192.168.79.21/drupal/index.php rather than http://192.168.79.21/drupal/index.php?q=civicrm/event/manage/registration

        Attachments

          Activity

          [CRM-8630] Event Tabs Broken without Drupal Clean URLs Enabled
          Donald A. Lobo added a comment -

          Hey paul:

          can you try the attacched patch. Please ensure you delete templates_c directory after applying the patch

          thanx

          paul (Inactive) added a comment -

          Hi - this doesn't quite work because the POST url becomes: /drupal/index.php?q=civicrm/event/manage/fee&reset=1 rather than /drupal/index.php?q=civicrm/event/manage/fee

          Paul

          Donald A. Lobo added a comment -


          second version of patch attached. can u give it another shot

          paul (Inactive) added a comment -

          Apart from the mistake on line 6 of the patch (you've lost a closing brace) it works perfectly!
          Thank you!

            People

            • Assignee:
              Donald A. Lobo
              Reporter:
              paul campbell

              Dates

              • Created:
                Updated:
                Resolved: