mahara-contributors team mailing list archive

[Mahara Bot <1457867@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/4792
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/805cc3521228f9fb2986d1f7b36d18eb34c768bb
Submitter: Aaron Wells (aaronw@xxxxxxxxxxxxxxx)
Branch:    15.04_STABLE

commit 805cc3521228f9fb2986d1f7b36d18eb34c768bb
Author: Wen-Chang Chien <wcchien@xxxxxxxxxxx>
Date:   Mon May 25 10:31:26 2015 +1200

Variable name typo breaks artefact export

Bug 1457867

Change-Id: Id9e18b0341589026bff1b67357ab56a667ef3b26
(cherry picked from commit 951ac53f2ce34d9dcc79d17ea6399b17f5f12a47)

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1457867

Title:
  Full-portfolio export doesn't include all artefacts

Status in Mahara ePortfolio:
  Fix Committed
Status in Mahara 15.04 series:
  Fix Committed
Status in Mahara 15.10 series:
  Fix Committed

Bug description:
  Mahara Ver. 15.04

  line 243 ~ 256 of  /htdocs/export/lib.php

  -------------------------------------------------------------------------------------
          // Get the list of artefacts to export
          if ($artefacts == self::EXPORT_ALL_ARTEFACTS) {
              $query = 'SELECT id FROM {artefact} WHERE "owner" = ?';
              $args = array($userid);
              if (sizeof($tmpviews)) {
                  $query .= 'UNION
                      SELECT artefact
                      FROM {view_artefact}
                  WHERE "view" IN (SELECT id FROM {view} WHERE "owner" = ?)
                      ORDER BY id';
                  $args[] = $userid;
                  $this->artefactexportmode = $tempartefacts = get_column_sql($query, $args);
              }
          }
  -------------------------------------------------------------------------------------

         $tempartefacts is a  is misspelled variable.

  Correct variable name is  $tmpartefacts.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1457867/+subscriptions