Moodle

META: Develop new Repository API

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 2.0
  • Fix Version/s: 2.0
  • Component/s: Repositories
  • Labels:
    None
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

This meta bug lists and links all the subtasks for developing the repository API which is specified here:

http://docs.moodle.org/en/Development:Repository_API

Some discussion and a large number of votes is on MDL-9134

Issue Links

Progress
Resolved Sub-Tasks Unresolved Sub-Tasks

Sub-Tasks

Activity

Hide
Nicolas Connault added a comment -

Attached a proposed refactoring of repo code.

Show
Nicolas Connault added a comment - Attached a proposed refactoring of repo code.
Hide
Petr Škoda (skodak) added a comment -

the latest commit http://cvs.moodle.org/moodle/repository/mahara/repository.class.php?r1=1.5&r2=1.6
does not seem correct, there is probably problem with $PAGE not pixpath, in any case pixpath is mandatory, you should not work around it

Show
Petr Škoda (skodak) added a comment - the latest commit http://cvs.moodle.org/moodle/repository/mahara/repository.class.php?r1=1.5&r2=1.6 does not seem correct, there is probably problem with $PAGE not pixpath, in any case pixpath is mandatory, you should not work around it
Hide
Dongsheng Cai added a comment -

I talked to Time about this issue before, the problem is, in web service, there is no moodle_page existed (exchanging json string only), do we really need to setup a useless $PAGE object just in order to use pixpath?

Show
Dongsheng Cai added a comment - I talked to Time about this issue before, the problem is, in web service, there is no moodle_page existed (exchanging json string only), do we really need to setup a useless $PAGE object just in order to use pixpath?
Hide
Leo Thiessen added a comment -

MrCUTE Jr. is a simple version of a shared url/file repository that currently works for moodle 1.9.5 to provide a quick solution for shared resource needs. It provides a search box to find the shared resources you are looking for based on kewords and other data associated with the shared resource when it was entered into the MrCUTE Jr. repository.

I flagged this as "duplicates" the repository api, based on it being the nearest match; but it doesn't do what the repository api does - rather provides a function that the repository api will enable, more fully, in the future. I hope this will solve some simpler needs for Moodle 1.9.5 users in the meantime.

Feedback is welcomed, please!

Show
Leo Thiessen added a comment - MrCUTE Jr. is a simple version of a shared url/file repository that currently works for moodle 1.9.5 to provide a quick solution for shared resource needs. It provides a search box to find the shared resources you are looking for based on kewords and other data associated with the shared resource when it was entered into the MrCUTE Jr. repository. I flagged this as "duplicates" the repository api, based on it being the nearest match; but it doesn't do what the repository api does - rather provides a function that the repository api will enable, more fully, in the future. I hope this will solve some simpler needs for Moodle 1.9.5 users in the meantime. Feedback is welcomed, please!
Hide
Nick Thompson added a comment -

Maybe I'm missing something, but according to the specs found here:
http://docs.moodle.org/en/Development:Repository_API

For every file I should be able to choose to copy the file into moodle, or link to it externally.
"only return the URL to the file if it's desired to keep it external"

I don't see that option anywhere...

Show
Nick Thompson added a comment - Maybe I'm missing something, but according to the specs found here: http://docs.moodle.org/en/Development:Repository_API For every file I should be able to choose to copy the file into moodle, or link to it externally. "only return the URL to the file if it's desired to keep it external" I don't see that option anywhere...
Hide
Martin Dougiamas added a comment -

Nick, see MDL-20602. It's not in CVS yet.

Show
Martin Dougiamas added a comment - Nick, see MDL-20602. It's not in CVS yet.
Hide
Martin Dougiamas added a comment -

Resolving the main bug as repositories are clearly in and working. Will let the few subtasks take care of themselves.

Thanks Dongsheng and Petr and others for all your work on this!

Show
Martin Dougiamas added a comment - Resolving the main bug as repositories are clearly in and working. Will let the few subtasks take care of themselves. Thanks Dongsheng and Petr and others for all your work on this!

Dates

  • Created:
    Updated:
    Resolved: