MDL-49490 assign: can't export to portfolio on non-submission

buitranduong · Nov 18, 2015 · a49fca4 · a49fca4
1 parent 26653c8
commit a49fca4
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/mod/assign/renderable.php b/mod/assign/renderable.php
@@ -788,7 +788,7 @@ public function __construct(context $context, $sid, $filearea, $component) {
 
         if (!empty($CFG->enableportfolios)) {
             require_once($CFG->libdir . '/portfoliolib.php');
-            if (count($files) >= 1 &&
+            if (count($files) >= 1 && !empty($sid) &&
                     has_capability('mod/assign:exportownsubmission', $this->context)) {
                 $button = new portfolio_add_button();
                 $callbackparams = array('cmid' => $this->cm->id,
@@ -823,6 +823,7 @@ public function preprocess($dir, $filearea, $component) {
         foreach ($dir['files'] as $file) {
             $file->portfoliobutton = '';
             if (!empty($CFG->enableportfolios)) {
+                require_once($CFG->libdir . '/portfoliolib.php');
                 $button = new portfolio_add_button();
                 if (has_capability('mod/assign:exportownsubmission', $this->context)) {
                     $portfolioparams = array('cmid' => $this->cm->id, 'fileid' => $file->get_id());