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

Cannot translate Price set options on Edit option page

    Details

    • Documentation Required?:
      None

      Description

      Attempting to click on the little green and grey translate icon generates a network error. Looking in the logs indicates that the ajax callback is generating a database syntax error because the link does not contain an id for the price set option.

      In the template file CRM/Price/Form/Option.tpl the missing variable is $sid which is never initialized. I verified that it is never initialized by using &smartyDebug=1 as well as by looking at all the $this->assign calls in CRM/Price/Form/Option.php and it's parents.

      I also believe that the variable is not the correct one. It should in fact be using $optionId in the translation popup links.

      I believe this affects the 4.4 series as well since this section of code has not changed. We also experienced this on 4.4 but I haven't retested recently.

      I will submit a patch or if I have additional time a pull request.

        Attachments

          Activity

            People

            • Assignee:
              colemanw Coleman Watts
              Reporter:
              lola_slade Lola Slade
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: