Merge branch 'MDL-35983_equella_filename_dev' of git://github.com/dongsheng/moodle
authorDan Poltawski <dan@moodle.com>
Mon, 8 Dec 2014 14:23:49 +0000 (14:23 +0000)
committerDan Poltawski <dan@moodle.com>
Mon, 8 Dec 2014 14:23:49 +0000 (14:23 +0000)
1  2 
repository/equella/callback.php

@@@ -17,7 -17,7 +17,7 @@@
  /**
   * Callback for equella repository.
   *
 - * @since 2.3
 + * @since Moodle 2.3
   * @package   repository_equella
   * @copyright 2012 Dongsheng Cai {@link http://dongsheng.org}
   * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
@@@ -36,7 -36,11 +36,11 @@@ if (isset($info->url)) 
  }
  
  $filename = '';
- if (isset($info->name)) {
+ // Use $info->filename if exists, $info->name is a display name,
+ // it may not have extension
+ if (isset($info->filename)) {
+     $filename  = s(clean_param($info->filename, PARAM_FILE));
+ } else if (isset($info->name)) {
      $filename  = s(clean_param($info->name, PARAM_FILE));
  }
  
@@@ -55,7 -59,7 +59,7 @@@ if (isset($info->license)) 
      $license = s(clean_param($info->license, PARAM_ALPHAEXT));
  }
  
 -$source = base64_encode(serialize((object)array('url'=>$url,'filename'=>$filename)));
 +$source = base64_encode(json_encode(array('url'=>$url,'filename'=>$filename)));
  
  $js =<<<EOD
  <html>