CRM-8328 Clicking the civicrm/extern/url.php links on a CiviMail permalink page gives "Missing input parameters"

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 3.4.2
    • Fix Version/s: 3.4.4, 4.0.4
    • Component/s: CiviMail
    • Labels:
      None

      Description

      See forum post http://forum.civicrm.org/index.php/topic,20360.0.html

      In a CiviMail, the links of the form .../civicrm/extern/url.php work correctly since both the 'u' and 'qid' parameters are supplied.

      However, if I visit the permalink page, those links fail since only the 'u' parameter is supplied.

      The solution options are to either get civicrm/extern/url.php to work without the 'qid' parameter, or to somehow supply it. I don't think the second option is valid since permalink pages can be made publicly visible, and 'qid' is not meaningful in that case.

      In the forum thread I suggest a fix which does the following if 'qid' is not present ...

      • retrieve the URL solely on the basis of the 'u' parameter
      • skips the creation of the "click through" event

      I haven't yet tested that fix.

        Attachments

          Activity

          [CRM-8328] Clicking the civicrm/extern/url.php links on a CiviMail permalink page gives "Missing input parameters"
          Ken West added a comment -

          Doh! I forgot to set the Priority on this issue: I reckon it's more than Trivial. It renders the permalink page useless if (some of) the links on it don't work.

          Ken West added a comment -

          BTW, I wrote about two options, but I guess we can have both.

          • apply a fix like the one I suggested
          • add a 'qid' parameter to the permalink URL in the CiviMail so civicrm/mailing/view can create a page where each of the civicrm/extern/url.php links has the 'qid' parameter
          Ken West added a comment -

          I've tested this both with a qid and without.

          Rohan S. Chavan added a comment -

          Tested in r34945

            People

            • Assignee:
              Rohan S. Chavan
              Reporter:
              Ken West

              Dates

              • Created:
                Updated:
                Resolved: