Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-16512

Activation email is sent even if user registration fails (patch)

    Details

      Description

      If the user object is stored, but the publishing fails, this is not detected. The activation email will be sent, and will appear OK, since it takes its content from the user object. The user node however does not exist, so the user can't login and the admin can't find the user by searching.

      The patch solves this by stopping execution when publishing fails. The standard warning "Unable to register new user" (already existing in templates) will be shown.

      1. user-register-fail.diff
        2 kB
        (inactive) Gunnstein Lye

        Activity

        Show
        (inactive) Gunnstein Lye added a comment - Fixed in master (4.4.0beta2) http://github.com/ezsystems/ezpublish/commit/2a9f0b7e821f94ac2094b9976e12061ee750c4f8 stable-4.3 (4.3.1) http://github.com/ezsystems/ezpublish/commit/48b53d81003fcf49649b06a8dbb469339b9187f1 stable-4.2 (4.2.1) http://github.com/ezsystems/ezpublish/commit/4c08096da87b1e317f302de83130f7a9ccefee95
        Hide
        Patrick Allaert added a comment -

        Commit https://github.com/ezsystems/ezpublish/commit/4c08096da87b1e317f302de83130f7a9ccefee95 in stable-4.2 contains calls to ezpI18n which only exists in 4.3+

        Show
        Patrick Allaert added a comment - Commit https://github.com/ezsystems/ezpublish/commit/4c08096da87b1e317f302de83130f7a9ccefee95 in stable-4.2 contains calls to ezpI18n which only exists in 4.3+
        Hide
        (inactive) Gunnstein Lye added a comment -

        In reply to comment #051299
        Patrick, Bertrand,
        Sorry, my mistake. I'll fix this unless Bertrand is already on it? (He's the issue owner.)

        Show
        (inactive) Gunnstein Lye added a comment - In reply to comment #051299 Patrick, Bertrand, Sorry, my mistake. I'll fix this unless Bertrand is already on it? (He's the issue owner.)
        Hide
        (inactive) Gunnstein Lye added a comment -

        In reply to comment #051300
        Thanks Patrick, I saw you fixed it.

        Show
        (inactive) Gunnstein Lye added a comment - In reply to comment #051300 Thanks Patrick, I saw you fixed it.
        Hide
        André R added a comment -
        Show
        André R added a comment - Re closing, regression fixed by Patrick in: https://github.com/ezsystems/ezpublish/commit/4d443028fbfad0ae546a379246f07d48ac9843c9
        Hide
        Geir Arne Waaler added a comment -

        Reported as fixed. I move it to Closed.

        Geir Arne Waaler
        eZ Documentation

        Show
        Geir Arne Waaler added a comment - Reported as fixed. I move it to Closed. Geir Arne Waaler eZ Documentation

          People

          • Assignee:
            Bertrand Dunogier
            Reporter:
            (inactive) Gunnstein Lye
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: