Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.5.3
    • Fix Version/s: None
    • Component/s: Language
    • Labels:
      None
    • Environment:
      All
    • Database:
      Any
    • Affected Branches:
      MOODLE_15_STABLE

      Description

      Script: login/signup.php

      PHP E_ALL

      Notice: Undefined property: lastname in C:\Programme\wampserver\www\mvhs\lib\moodlelib.php(4460) : eval()'d code on line 1

      Notice: Undefined property: lastname in C:\Programme\wampserver\www\mvhs\lib\moodlelib.php(4460) : eval()'d code on line 1

      Reason: lib/moodlelib.php uses $a->firstname as fullname.

      $a->lastname does not exist and is not defined (line 3615).

      FIX: language file de/moodle.php - for $string['emailconfirmation'] delete $a->lastname.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            dougiamas Martin Dougiamas added a comment -

            From oen roggemans (koen at roggemans.net) Wednesday, 1 March 2006, 10:25 PM:

            Hm, no, must not be deleted. It must be something different

            That string is meant to display the lastname too.

            Do your user fields contain last names?

            From Tobias (th at hauser-wenz.de) Wednesday, 1 March 2006, 10:41 PM:

            Yes, they do. I have signed up a new user with lastname. And the global configuration for fullnamedisplay is firstname plus lastname.

            I have looked at other language packs and there the $string['emailconfirmation'] does only contain the firstname. I found one place in lib/moodlelib.php where the firstname is set to fullname (line 3645) - I presume, thats the problem - and I var_dump-ed the $result-string in line 4460. That produced a string with $a->lastname out of the language pack vor the eval(). Therefore the conclusion, to change the $string['emailconfirmation'] in the de-language pack.

            From oen roggemans (koen at roggemans.net) Wednesday, 1 March 2006, 10:56 PM:

            Yes, now I see. It is an error in the de langpack.

            I think Ralph (as I remember correct) wanted to have first name and lastname displayed there. I'll contact him to take a look at it and keep you informed.

            Thank you for reporting

            From dialoge dialoge (rh at dialoge.net) Wednesday, 1 March 2006, 11:51 PM:

            Yes it is correct. We have to delete this string in the emailconfirmation.

            I remember when we changed this, it was the correct way. Perhaps there are changes in the use of first-/lastname in the code.

            Koen, can you change it in the german file. I can't do it in the next days.

            Show
            dougiamas Martin Dougiamas added a comment - From oen roggemans (koen at roggemans.net) Wednesday, 1 March 2006, 10:25 PM: Hm, no, must not be deleted. It must be something different That string is meant to display the lastname too. Do your user fields contain last names? From Tobias (th at hauser-wenz.de) Wednesday, 1 March 2006, 10:41 PM: Yes, they do. I have signed up a new user with lastname. And the global configuration for fullnamedisplay is firstname plus lastname. I have looked at other language packs and there the $string ['emailconfirmation'] does only contain the firstname. I found one place in lib/moodlelib.php where the firstname is set to fullname (line 3645) - I presume, thats the problem - and I var_dump-ed the $result-string in line 4460. That produced a string with $a->lastname out of the language pack vor the eval(). Therefore the conclusion, to change the $string ['emailconfirmation'] in the de-language pack. From oen roggemans (koen at roggemans.net) Wednesday, 1 March 2006, 10:56 PM: Yes, now I see. It is an error in the de langpack. I think Ralph (as I remember correct) wanted to have first name and lastname displayed there. I'll contact him to take a look at it and keep you informed. Thank you for reporting From dialoge dialoge (rh at dialoge.net) Wednesday, 1 March 2006, 11:51 PM: Yes it is correct. We have to delete this string in the emailconfirmation. I remember when we changed this, it was the correct way. Perhaps there are changes in the use of first-/lastname in the code. Koen, can you change it in the german file. I can't do it in the next days.
            Hide
            mblake Michael Blake added a comment -

            assign to a valid user

            Show
            mblake Michael Blake added a comment - assign to a valid user
            Hide
            mblake Michael Blake added a comment -

            assign to a valid user

            Show
            mblake Michael Blake added a comment - assign to a valid user

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: