[#MDL-8330] TTW language updater removes language CVS checkout and thus local modification

Issue Details (XML | Word | Printable)

Key:	MDL-8330
Type:	Bug
Status:	Closed
Resolution:	Fixed
Priority:	Minor
Assignee:	Yu Zhang
Reporter:	David Mudrak
Votes:	1
Watchers:	3

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Moodle

TTW language updater removes language CVS checkout and thus local modification

Created: 28/Jan/07 11:01 PM Updated: 01/Nov/08 05:33 AM

Return to search

Component/s:

Languages

Affects Version/s:

1.8

Fix Version/s:

1.8.4, 1.9, 2.0

File Attachments:

MDL-8330.patch (2 kB)

Issue Links:

Duplicate

This issue is duplicated by:

~~MDL-10227~~ Update overwrites Non en-utf8 langfiles

Relates

This issue has a non-specific relationship to:

~~MDL-17094~~ Allow translators to disable automatic language update during Moodle upgrade

Participants:	David Mudrak, Koen Roggemans, Nicolas Martignoni and Yu Zhang
Security Level:	None
QA Assignee:	Nicolas Martignoni
Resolved date:	15/Nov/07
Affected Branches:	MOODLE_18_STABLE
Fixed Branches:	MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

Description

« Hide

As a translator, I have got CVS checkout of both HEAD source code (CVS module "moodle") and HEAD languages (module "lang"). I regularly run "cvs update" in both working copies and I sometimes do "cvs checkin" in my language pack.

But - if I have to upgrade my dev installation because of the version shift, the first step appears to be an automatic language pack update (TTW - thru the web). This tool probably removes my own CVS checkout and it downloads and unpack standard lang pack from moodle.org.

From my point of view, this is a bug. The tool should check, if the language pack is CVS working copy. If it is so, the automatic update should not proceed, maybe with notification to run "cvs update" in language directory.

All

Comments

Change History

Version Control

Sort Order:

[ Permalink | « Hide ]

Nicolas Martignoni added a comment - 13/Feb/07 03:58 AM

Couldn't this be resolved (at least partially) by resolving MDL-6645?

[ Permalink | « Hide ]

Koen Roggemans added a comment - 09/Mar/07 06:25 AM

Hi Nicolas
If I understand David correctly, I don't think it is related.

Is this not a translator-only problem? Personally I never noticed it. Probably because I don't do version shifts, but have a test installation for each version.
Isn't your problem solved by first updating your installation and then updating your language packs or do I understand this completely wrong?

[ Permalink | « Hide ]

Nicolas Martignoni added a comment - 09/Mar/07 03:07 PM

Hi Koen,
You're probably right. Neither did I suffer from this myself, and I think too that this is not related too MDL-6645.

Suggesting resolving as not reproducible, unless reporter experiment it still now.

[ Permalink | « Hide ]

Koen Roggemans added a comment - 17/Mar/07 06:23 AM

Assume it is ok to close this now

[ Permalink | « Hide ]

Nicolas Martignoni added a comment - 17/Mar/07 06:25 PM

Closing, as not reproducible.

[ Permalink | « Hide ]

David Mudrak added a comment - 05/May/07 11:57 PM

Sorry guys, but I have to reopen this issue again. See http://moodle.org/mod/forum/discuss.php?d=70849
You are right, this is just translators' problem. But as long as this issue may lead to accidental translated string lost, I vote for fix. Or, at least, upgrade script should produce some warnings about removing the current language pack and confirmation should be needed.

Here are steps to reproduce:
1. I suppose you have a 1.8 installation. Do some modifications in a datamoodle/lang/xxx_utf8 package as a translator (e.g. by admin/lang.php)
2. Imagine you forget to check in these modifications, send them by email to the pack maintainer or just want to continue later.
3. Proceed a Moodle upgrade, i.e. overwrite source codes with new version and go to /admin
4. Please note - the language pack are automatically downloaded and you version is SILENTLY removed.

So, without any warning, your translations are lost just because you forgot to backup your lang/xxx_utf8... Is the problem more clear now?

[ Permalink | « Hide ]

Nicolas Martignoni added a comment - 06/May/07 06:30 PM

Yes, David, thank you for the clarification.

Confirming. And my +1 for resolution.

[ Permalink | « Hide ]

Koen Roggemans added a comment - 09/May/07 06:01 AM

Assigned this to Martin.
Can we have a warning or maybe better: the possibility not to upgrade the language packs while doing an upgrade?

[ Permalink | « Hide ]

Nicolas Martignoni added a comment - 09/May/07 02:33 PM

Like Koen, I would find better not to upgrade the lang packs while doing an upgrade.

Another option would be to let the admin choose if he does want the lang update or not.

[ Permalink | « Hide ]

David Mudrak added a comment - 24/Jul/07 12:49 AM

See comments in ~~MDL-10227~~

[ Permalink | « Hide ]

David Mudrak added a comment - 15/Nov/07 04:38 AM

Several possible solutions have been proposed here and at ~~MDL-10227~~. IMO it is enough to just put a warning message for translators that their modifications might be overwritten. See the attached ~~MDL-8330~~.patch introducing such a message.

[ Permalink | « Hide ]

Nicolas Martignoni added a comment - 15/Nov/07 05:14 AM

David, I think your solution is at least a progress in regard of the current situation. My +1 (if it counts) for it.
Thanks.

[ Permalink | « Hide ]

Yu Zhang added a comment - 15/Nov/07 09:21 AM

Patch in 1.8, 1.9 and HEAD, thanks!

[ Permalink | « Hide ]

Nicolas Martignoni added a comment - 21/Nov/07 04:53 PM

Verified, closing. Thanks!

All content on this web site is made available under the GNU General Public License, unless otherwise stated.