Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.3
    • Component/s: Blog, Language
    • Labels:

      Description

      There are some places where the term BLOG is used when BLOG POST is meant. (In much the same way FORUM needs to be distinguished from FORUM POST)
      eg in delete > delete blog (se picture)

        Gliffy Diagrams

          Activity

          Hide
          Helen Foster added a comment -

          Derek, thanks for your report. Just wondering whether we should use the term 'blog post' or 'blog entry'?

          Show
          Helen Foster added a comment - Derek, thanks for your report. Just wondering whether we should use the term 'blog post' or 'blog entry'?
          Hide
          Derek Chirnside added a comment -

          Hmm.
          Forum Post + Blog post seems more consistent.
          On balance at the moment I prefer post.
          But in actual fact I don't mind.

          Show
          Derek Chirnside added a comment - Hmm. Forum Post + Blog post seems more consistent. On balance at the moment I prefer post. But in actual fact I don't mind.
          Hide
          Derek Chirnside added a comment -

          this is another place where there is ambiguity. When you click on Blogs, you see the global list of all blog posts for the site.

          Show
          Derek Chirnside added a comment - this is another place where there is ambiguity. When you click on Blogs, you see the global list of all blog posts for the site.
          Hide
          Jason Fowler added a comment -

          I think entry is a more appropriate term, after all, you make an entry in a log, and the blog is just a weblog...

          Show
          Jason Fowler added a comment - I think entry is a more appropriate term, after all, you make an entry in a log, and the blog is just a weblog...
          Hide
          Jason Fowler added a comment -

          Just changed the string for the entry delete confirmation string - the nav is okay in my opinion, blogs are a collection of entries, which is exactly what the Site blogs page displays.

          Show
          Jason Fowler added a comment - Just changed the string for the entry delete confirmation string - the nav is okay in my opinion, blogs are a collection of entries, which is exactly what the Site blogs page displays.
          Hide
          Dan Poltawski added a comment -

          Basically the same as my comments on MDL-26014:

          Our translators have a real problem keeping up with the number of strings which we introduce so we should try and clean up unused strings.

          So, is the existing string used anywhere else? If not then lets:

          a) 'Rename' the string identifier
          1) Delete the url string
          2) Do an AMOS mov so that other languages (which may have translated that field better than english) us don't have to retranslate. (
          http://docs.moodle.org/dev/Languages/AMOS#AMOS_script)

          or

          b) Leave the string identifier as it is and just change the string in place

          Show
          Dan Poltawski added a comment - Basically the same as my comments on MDL-26014 : Our translators have a real problem keeping up with the number of strings which we introduce so we should try and clean up unused strings. So, is the existing string used anywhere else? If not then lets: a) 'Rename' the string identifier 1) Delete the url string 2) Do an AMOS mov so that other languages (which may have translated that field better than english) us don't have to retranslate. ( http://docs.moodle.org/dev/Languages/AMOS#AMOS_script ) or b) Leave the string identifier as it is and just change the string in place
          Hide
          Jason Fowler added a comment -

          Just replaced the string, after a quick check of where else it was used.

          Show
          Jason Fowler added a comment - Just replaced the string, after a quick check of where else it was used.
          Hide
          Dan Poltawski added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Jason Fowler added a comment -

          this has been rebased

          Show
          Jason Fowler added a comment - this has been rebased
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated (master), thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated (master), thanks!
          Hide
          Ankit Agarwal added a comment -

          works as expected!!
          Passing!

          Show
          Ankit Agarwal added a comment - works as expected!! Passing!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This has been near becoming rejected, because it's not the best code you are able to produce.

          But, luckily, at the end, it has landed and has been spread to all repos out there.

          Many thanks and, don't forget it, keep improving your skills, you can!

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - This has been near becoming rejected, because it's not the best code you are able to produce. But, luckily, at the end, it has landed and has been spread to all repos out there. Many thanks and, don't forget it, keep improving your skills, you can! Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: