Moodle
  1. Moodle
  2. MDL-37986

Upgrade "Invalid installation files detected" message to remove CVS indications

        Details

        • Type: Improvement Improvement
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.3.4, 2.4, 2.5
        • Fix Version/s: None
        • Component/s: Installation, Language
        • Labels:
        • Affected Branches:
          MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE

          Description

          CVS servers have been disconnected, so i think we should remove references to CVS in core_admin's upgradestalefilesinfo message.

            Gliffy Diagrams

              Issue Links

              has been marked as being related by

              Improvement - An enhancement to an existing Moodle feature. MDLSITE-1363 Create a way for translators and QA people to easily fix the English pack

              • Minor - Minor loss of function where workaround is possible
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Séverin Terrier added a comment -

                Changes made for Moodle 2.3 to 2.5 in http://lang.moodle.org/local/amos/contrib.php?id=5693

                Show
                Séverin Terrier added a comment - Changes made for Moodle 2.3 to 2.5 in http://lang.moodle.org/local/amos/contrib.php?id=5693
                Hide
                Permalink
                Séverin Terrier added a comment -

                We should perhaps also remove in local_moodleorg.php the strings 'downloadcvs' and 'downloadcvstitle'...

                Show
                Séverin Terrier added a comment - We should perhaps also remove in local_moodleorg.php the strings 'downloadcvs' and 'downloadcvstitle'...
                Hide
                Permalink
                Séverin Terrier added a comment -

                I've also started a discussion about removing CVS informations :
                https://moodle.org/mod/forum/discuss.php?d=225052

                Show
                Séverin Terrier added a comment - I've also started a discussion about removing CVS informations : https://moodle.org/mod/forum/discuss.php?d=225052
                Hide
                Permalink
                Helen Foster added a comment -

                Séverin, thanks for your report and your AMOS contribution. I have committed the changes to en_fix, and will be available in the en lang pack when en_fix is merged with it within a month.

                David, I've added you as a watcher, hoping you can help with removing strings from local_moodleorg.php.

                Show
                Helen Foster added a comment - Séverin, thanks for your report and your AMOS contribution. I have committed the changes to en_fix, and will be available in the en lang pack when en_fix is merged with it within a month. David, I've added you as a watcher, hoping you can help with removing strings from local_moodleorg.php.
                Hide
                Permalink
                Dan Poltawski added a comment -

                Hmm, i'm not quite sure how local_moodleorg strings are getting into amos at the moment.

                Show
                Dan Poltawski added a comment - Hmm, i'm not quite sure how local_moodleorg strings are getting into amos at the moment.
                Hide
                Permalink
                David Mudrak added a comment -

                IIRC local_moodleorg is being imported into AMOS manually by myself per-demand. It's the same for all plugins managed by HQ that are not being published in moodle.org/plugins.

                Show
                David Mudrak added a comment - IIRC local_moodleorg is being imported into AMOS manually by myself per-demand. It's the same for all plugins managed by HQ that are not being published in moodle.org/plugins.
                Hide
                Permalink
                Helen Foster added a comment -

                David, please could you resolve this issue when the strings are removed from local_moodleorg.

                Show
                Helen Foster added a comment - David, please could you resolve this issue when the strings are removed from local_moodleorg.
                Hide
                Permalink
                David Mudrak added a comment -

                This issue was assigned to me, 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
                David Mudrak added a comment - This issue was assigned to me, 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
                Hide
                Permalink
                Petr Skoda added a comment -

                I did not find any CVS leftovers in master or local_moodleorg repo, closing. thanks

                Show
                Petr Skoda added a comment - I did not find any CVS leftovers in master or local_moodleorg repo, closing. thanks
                Hide
                Permalink
                Séverin Terrier added a comment -

                For what i can see in AMOS, local_moodleorg is for 2.3 ONLY, and still have the strings 'downloadcvs' and 'downloadcvstitle'. Don't know if it's correct/usefull/useless...

                Show
                Séverin Terrier added a comment - For what i can see in AMOS, local_moodleorg is for 2.3 ONLY, and still have the strings 'downloadcvs' and 'downloadcvstitle'. Don't know if it's correct/usefull/useless...
                Hide
                Permalink
                David Mudrak added a comment -

                The "downloadviagitcvs" string was replaced with the "downloadviagit" one by Dan on Mon Jan 21 08:49:27 2013 +0800 in the local_moodleorg plugin repository. The "downloadcvstitle" string was removed by Dan on Thu Mar 22 11:03:09 2012 +0800 there. So the only thing is that the plugin's strings were not updated in AMOS for a long time. I am going to do it now.

                Show
                David Mudrak added a comment - The "downloadviagitcvs" string was replaced with the "downloadviagit" one by Dan on Mon Jan 21 08:49:27 2013 +0800 in the local_moodleorg plugin repository. The "downloadcvstitle" string was removed by Dan on Thu Mar 22 11:03:09 2012 +0800 there. So the only thing is that the plugin's strings were not updated in AMOS for a long time. I am going to do it now.
                Hide
                Permalink
                David Mudrak added a comment -

                Done. The "local_moodleorg" component now appears to be at the 2.5 branch and it has the current strings registered only.

                Show
                David Mudrak added a comment - Done. The "local_moodleorg" component now appears to be at the 2.5 branch and it has the current strings registered only.
                Hide
                Permalink
                Séverin Terrier added a comment -

                Thanks David, it's nice like that

                Show
                Séverin Terrier added a comment - Thanks David, it's nice like that

                  People

                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    4 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: