Merge branch 'MDL-52318-master' of https://github.com/snake/moodle
authorAndrew Nicols <andrew@nicols.co.uk>
Wed, 11 Jul 2018 00:33:13 +0000 (08:33 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Wed, 11 Jul 2018 00:33:13 +0000 (08:33 +0800)
1  2 
lib/form/url.php

diff --combined lib/form/url.php
@@@ -48,6 -48,9 +48,9 @@@ class MoodleQuickForm_url extends HTML_
      /** @var bool if true label will be hidden */
      var $_hiddenLabel=false;
  
+     /** @var string the unique id of the filepicker, if enabled.*/
+     protected $filepickeruniqueid;
      /**
       * Constructor
       *
  
          // Print out file picker.
          $str .= $this->getFilePickerHTML();
+         $str = '<div id="url-wrapper-' . $this->get_filepicker_unique_id() . '">' . $str . '</div>';
  
          return $str;
      }
          global $PAGE, $OUTPUT;
  
          $str = '';
-         $clientid = uniqid();
+         $clientid = $this->get_filepicker_unique_id();
  
          $args = new stdClass();
          $args->accepted_types = '*';
          if (count($options->repositories) > 0) {
              $straddlink = get_string('choosealink', 'repository');
              $str .= <<<EOD
 -<button id="filepicker-button-js-{$clientid}" class="visibleifjs btn btn-secondary">
 +<button type="button" id="filepicker-button-js-{$clientid}" class="visibleifjs btn btn-secondary">
  $straddlink
  </button>
  EOD;
      public function export_for_template(renderer_base $output) {
          $context = $this->export_for_template_base($output);
          $context['filepickerhtml'] = !empty($this->_options['usefilepicker']) ? $this->getFilePickerHTML() : '';
+         // This will conditionally wrap the element in a div which can be accessed in the DOM using the unique id,
+         // and allows the filepicker callback to find its respective url field, if multiple URLs are used.
+         if ($this->_options['usefilepicker']) {
+             $context['filepickerclientid'] = $this->get_filepicker_unique_id();
+         }
          return $context;
      }
  
          return true;
      }
  
+     /**
+      * Returns the unique id of the file picker associated with this url element, setting it in the process if not set.
+      *
+      * @return string the unique id of the file picker.
+      */
+     protected function get_filepicker_unique_id() : string {
+         if (empty($this->filepickeruniqueid)) {
+             $this->filepickeruniqueid = uniqid();
+         }
+         return $this->filepickeruniqueid;
+     }
  }