Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-34779

Wrong token in Site Policies "full name format"?

          Details

            Description

            In Site policies the "Full name format" setting default is "Language" but it seems to be mistake.It should be "First name +Surname" as written in the description - default value.
            See image added here. it should be different string token?
            Correct me if I'm wrong.
            Thanks

              Gliffy Diagrams

                Attachments

                  Issue Links

                  is a regression caused by

                  Improvement - An enhancement to an existing Moodle feature. MDL-18824 fullnamedisplay default value should be 'Language'

                  • Major - Major loss of function, incorrect output
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    salvetore Michael de Raadt added a comment -

                    It looks like this default changed at some point but the associated language string was not updated.

                    Show
                    salvetore Michael de Raadt added a comment - It looks like this default changed at some point but the associated language string was not updated.
                    Hide
                    Permalink
                    rwijaya Rossiani Wijaya added a comment -

                    This looks good Ankit.

                    +1 for integration.

                    Show
                    rwijaya Rossiani Wijaya added a comment - This looks good Ankit. +1 for integration.
                    Hide
                    Permalink
                    ankit_frenz Ankit Agarwal added a comment -

                    Thanks for the review Rosie,
                    Sending for integration.

                    Show
                    ankit_frenz Ankit Agarwal added a comment - Thanks for the review Rosie, Sending for integration.
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    Hi Ankit,

                    I see you've correctly attributed the patch to Michael, but how about putting his lastname in the commit

                    cheers,
                    dan

                    Show
                    poltawski Dan Poltawski added a comment - Hi Ankit, I see you've correctly attributed the patch to Michael, but how about putting his lastname in the commit cheers, dan
                    Hide
                    Permalink
                    ankit_frenz Ankit Agarwal added a comment - - edited

                    updated

                    Show
                    ankit_frenz Ankit Agarwal added a comment - - edited updated
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    Integrated, thanks for that.

                    Show
                    poltawski Dan Poltawski added a comment - Integrated, thanks for that.
                    Hide
                    Permalink
                    rajeshtaneja Rajesh Taneja added a comment -

                    String displayed is as described.
                    Thanks for fixing this, Ankit.

                    Show
                    rajeshtaneja Rajesh Taneja added a comment - String displayed is as described. Thanks for fixing this, Ankit.
                    Hide
                    Permalink
                    stronk7 Eloy Lafuente (stronk7) added a comment -

                    YEAR!*

                    CAF*, TOT!*

                    • Your effort amazingly resulted. (unbelievable :-P)
                    • Closing as fixed.
                    • Tons of thanks.
                    Show
                    stronk7 Eloy Lafuente (stronk7) added a comment - YEAR!* CAF*, TOT!* Your effort amazingly resulted. (unbelievable :-P) Closing as fixed. Tons of thanks.

                      People

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

                        Dates

                        • Created:
                          Updated:
                          Resolved:
                          Fix Release Date:
                          10/Sep/12