Moodle

post: clean old backup/restore code

Details

  • Type: Sub-task Sub-task
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 2.0
  • Fix Version/s: STABLE backlog
  • Component/s: Backup
  • Labels:
  • Difficulty:
    Easy
  • Affected Branches:
    MOODLE_20_STABLE

Description

Once we have all the logic migrated to new architecture, we'll delete all the old backup/restore 1.x code (core, activities and blocks).

Issue Links

is blocked by

Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-22153 post: clean old backup/restore files once they aren't necessary anymore

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
David Mudrak added a comment -

There is

{modulename}

_get_participants() in all activity modules. It seems that only legacy backup/backuplib.php uses this function (I have not searched for contrib, though). We shall consider if this function is still part of standard module API and eventually remove it from NEWMODULE template or mark it as @deprecated

Show
David Mudrak added a comment - There is {modulename} _get_participants() in all activity modules. It seems that only legacy backup/backuplib.php uses this function (I have not searched for contrib, though). We shall consider if this function is still part of standard module API and eventually remove it from NEWMODULE template or mark it as @deprecated
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

While proceeding with the deletion of old files (MDL-22153), these potential candidates have been also found:

  • course format: xxxx_backup_format_data() and xxxx_restore_format_data()
  • activity: xxxx_refresh_events()

They need verification (together with xxxx_get_participants()) and be cleaned/deprecated.

Ciao

Show
Eloy Lafuente (stronk7) added a comment - While proceeding with the deletion of old files ( MDL-22153 ), these potential candidates have been also found: course format: xxxx_backup_format_data() and xxxx_restore_format_data() activity: xxxx_refresh_events() They need verification (together with xxxx_get_participants()) and be cleaned/deprecated. Ciao
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue.

For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment

Show
Eloy Lafuente (stronk7) added a comment - This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue. For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: