Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

so that others in the chain can still be tried (bug #536959)

Signed-off-by: Piers Harding <piers@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Evan Goldenberg <evang@catalyst.net.nz>
(cherry picked from commit 91ca5e74d354e48c64bec8928bd41fb660dcf6eb)

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

It turns out that the import code assumes that only one user will be imported
at a time and uses static variables in a few places.

By importing only one user per page load and redirecting back to the import
page, we can work-around this problem.

Signed-off-by: Francois Marier <francois@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

This was preventing entries like these from being imported:

   <entry>
        <title>Something</title>
        <id>portfolio:artefactinternal</id>
        <content type=""></content>
   </entry>

Signed-off-by: Francois Marier <francois@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Richard Mansfield <richardm@catalyst.net.nz>

Signed-off-by: Penny Leach <penny@mjollnir.org>

This is the second part of the fix for this bug.

Signed-off-by: Penny Leach <penny@mjollnir.org>

Signed-off-by: Penny Leach <penny@mjollnir.org>

This fixes the problem that just linked files (rather than linked file
entries) were not being imported.

Signed-off-by: Penny Leach <penny@mjollnir.org>

Since the blog is the default fallback importer, make it be aware of
selections and import them as blogs, rather than blogposts.

It's still necessary to have a strategy for importing these entries as
blogposts as well, in case some of the child entries have a higher
weighted strategy as the fallback, because in that case, they'll be
imported using that strategy and the selection would be lost.

Signed-off-by: Penny Leach <penny@mjollnir.org>

This is necessary as lots of places in Mahara throw exceptions with no
messages, as none are required, for example AccessDeniedException.

Signed-off-by: Penny Leach <penny@mjollnir.org>

This commit prevents the problem of multiple artefacts being created for
singular artefact types (eg contactinformation, firstname, etc)

This is not really needed in the Admin import UI, or over mnet, since
mnet categorically skips persondata entries, but it is definitely more
robust.

Signed-off-by: Penny Leach <penny@mjollnir.org>

Signed-off-by: Penny Leach <penny@mjollnir.org>

This is useful, for example, when you want to generate a random numeric
string

Signed-off-by: Penny Leach <penny@mjollnir.org>

Signed-off-by: Penny Leach <penny@mjollnir.org>

Signed-off-by: Penny Leach <penny@mjollnir.org>

this is useful for importing data over mnet, but also potentially in the
future when we can import leap data into existing user accounts

Signed-off-by: Penny Leach <penny@mjollnir.org>

File import plugin vs Leap import plugin
Local file upload import vs Imports over mnet

Signed-off-by: Penny Leach <penny@mjollnir.org>

- Breaking up of the validation into two stages:
    - Performed by the importtransport object, before moving files
      around
    - Performed by the import class, after the files are transported

To this end I changed the importtransport objects to take the
importqueue record (or a fake one) in their constructor, so they have
the same data available as the importer would.

Signed-off-by: Penny Leach <penny@mjollnir.org>

Signed-off-by: Penny Leach <penny@mjollnir.org>

This commit includes:

- Moved a lot of logic out of admin/users/add.php to the various plugins
- Moved around unzipping and detecting files into the transport layers
- Refactored a lot of transport code into the base class

I tested:

- Adding a user from a leap2a xml file
- Adding a user from a leap2a zip file
- Importing non leap (straight files) data over mnet

Signed-off-by: Penny Leach <penny@mjollnir.org>

- use the mahara logo if the user has no profile icon set
- use the user's personal website as the author uri, if set
- add a generator to indicate that the feed was generated by Mahara
- set the icon element to use favicon.ico
- set feed rights based on presence or lack of CC block in view
- display feed icon in the view if the view is public

(partial implementation of Eduforge Feature Request #3357)

Signed-off-by: Evan Goldenberg <evang@catalyst.net.nz>
Signed-off-by: Francois Marier <francois@catalyst.net.nz>

If uploading files for a group or an institution, do not include the user
quota.
(cherry picked from commit 45d57645244d8866a291101e70f8fe2fcf6e759c)

Signed-off-by: Evan Goldenberg <evang@catalyst.net.nz>

If a user has a quota which is lower than post_max_size or
upload_max_filesize, then this should be shown as the maximum upload size.

The actual checking of this is already handled by the upload function for
the pieform.
(cherry picked from commit af0facaa676e716a29163c79e112fda0e54817fb)

Signed-off-by: Evan Goldenberg <evang@catalyst.net.nz>

Signed-off-by: Evan Goldenberg <evang@catalyst.net.nz>

Importing a hundred users on Apache seems to take more than half a gig of RAM.

Signed-off-by: Francois Marier <francois@catalyst.net.nz>

Signed-off-by: Francois Marier <francois@catalyst.net.nz>

This change is to match what the bulkexport page generates.

Signed-off-by: Francois Marier <francois@catalyst.net.nz>