Issue Details (XML | Word | Printable)

Key: MDL-16280
Type: Sub-task Sub-task
Status: Open Open
Priority: Minor Minor
Assignee: Petr Škoda (skodak)
Reporter: Marc Grober
Votes: 2
Watchers: 3
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle
MDL-19521

Incorporate addition of dragmath features in adminlib.php

Created: 31/Aug/08 08:08 AM   Updated: 16/Jun/09 05:00 PM
Component/s: Unknown
Affects Version/s: 1.8.4, 1.8.5, 1.8.6, 1.9, 1.9.1, 1.9.2, 2.0
Fix Version/s: None

Participants: Marc Grober and Petr Škoda (skodak)
Security Level: None
Affected Branches: MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE


 Description  « Hide
As a result of some previous efforts dragmath is now offered in Moodle CVS, but relies on a version of adminlib.php that includes minimal setting for dragmath

'insertdragmath' => 'em.icon.dragmath.gif'

The problem is that as new releases of moodle are made available the dragmath cvs faces either having to provide a separate version for each release, or face possible inconsistencies in adminlib.php.

I don't think requiring or expecting that that all users have the most recent release is reasonable.

As Dragmath is expected to be included in core 2 moodle anyway and could be actually move to core 1.9 Moodle with very minimal work, I would like to see the core adminlib.php for version 1.8.4 on be altered in Stabel to include the additional code so that adminlib.php can be deleted from the dragmath cvs.



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Petr Škoda (skodak) added a comment - 10/Sep/08 05:12 AM
There are not going to be any more improvements in 1.8.x or backporting, only critical bug fixes. 1.9.x is not getting any new features either, only normal bugfixing - sometimes there are exceptions, but those are often made for security reasons or for better backwards compatibility.

If I understand it correctly, you want one plugin to work in 1.8.x, 1.9.x? This is not possible, maybe you could use branches in contrib, right?


Marc Grober added a comment - 10/Sep/08 05:32 AM
I am talking about one line in adminlib.php to support an app which was moved to CVS and as a result is presenting issues because it means that everytime someone changes adminlib.php a new version of adminlib.php has to be created for the app.... though in fact, unless I am asea on this, with 2.0 this one line will be included in adminlib.php and the added lines has no backwards compatibility issues....

Petr Škoda (skodak) added a comment - 10/Sep/08 05:37 AM
I do not understand at all what you want, could you please attach a patch?

Petr Škoda (skodak) added a comment - 10/Sep/08 05:51 AM
Found it, added the line and tested - now I can see one more checkbox there that does nothing, is that what you propose for 1.8.x and 1.9.x?

Marc Grober added a comment - 10/Sep/08 06:51 AM
It does nothing if dragmath is not installed, and invokes dragmath if dragmath is installed.
Arguably, the best solution would be to include the line but to have the item not selected. Then the user could be told to activate it from the admin gui, and no further patching of adminlib.php would ever be necessary....

Petr Škoda (skodak) added a comment - 10/Sep/08 07:02 AM
"It does nothing if dragmath is not installed" - the way I see it it adds a new option which a) does not work if the rest of dragmath not installed b) is not documented in 1.8.x and 1.9.x
many modifications require patching of files, people usually use patches and cvs to keep the after upgrades