Non-core contributed modules

Mod tables do not work with underscores in the module names

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.8, 1.9
  • Fix Version/s: 1.8, 1.9
  • Component/s: Package: ILP
  • Labels:
    None
  • Database:
    Any
  • Affected Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE

Description

Mod tables do not work with underscores in the module names

This is due to the $add parameter being PARAM_ALPHA.

Currently seeking new advice on naming structure.

Issue Links

Activity

Hide
James Ballard added a comment -

Mods to be renamed to:

mod/ilptarget
mod/ilpconcern

Show
James Ballard added a comment - Mods to be renamed to: mod/ilptarget mod/ilpconcern
Hide
Anthony Borrow added a comment -

James - I have created the new folders ilptarget and ilpconcern in /contrib/plugins/mod/ as discussed. This removes the problematic underscore and for consistency uses the singular form. Hopefully Moodle 2.0 will allow for the underscore to be used in module names (just for the sake of consistency in the way the underscore is used). To that end, I have created MDL-14640. Again, my apologies for misleading you on the naming and thanks for your patience and flexibility in resolving these issues. Peace - Anthony

Show
Anthony Borrow added a comment - James - I have created the new folders ilptarget and ilpconcern in /contrib/plugins/mod/ as discussed. This removes the problematic underscore and for consistency uses the singular form. Hopefully Moodle 2.0 will allow for the underscore to be used in module names (just for the sake of consistency in the way the underscore is used). To that end, I have created MDL-14640. Again, my apologies for misleading you on the naming and thanks for your patience and flexibility in resolving these issues. Peace - Anthony
Hide
James Ballard added a comment -

Modules have been renamed to

mod/ilptarget
mod/ilpconcern

Further documentation:
http://moodle.ulcc.ac.uk/mod/wiki/view.php?id=2674&page=CVS+update

CVS modules should now function correctly.

Show
James Ballard added a comment - Modules have been renamed to mod/ilptarget mod/ilpconcern Further documentation: http://moodle.ulcc.ac.uk/mod/wiki/view.php?id=2674&page=CVS+update CVS modules should now function correctly.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: