Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.2, 2.2, 2.3
    • Fix Version/s: 2.1.4, 2.2.1
    • Component/s: Messages
    • Labels:

      Description

      There are two ways to suspend user accounts:
      1/ user.auth==nologin
      2/ user.suspended==1

      All messaging plugins need to be fixed, at the same time we could prevent messaging to deleted users too.

      OH! Please note that word "messaging" here does not mean the direct sending of messages between users in moodle, it is fine to send messages to users this way, they just do not get notification via email/jabber/etc.

        Gliffy Diagrams

          Issue Links

            Activity

            Petr Skoda created issue -
            Petr Skoda made changes -
            Field Original Value New Value
            Description There are two ways to suspend user accounts:
            1/ user.auth==nologin
            2/ user.suspended==1

            All messaging plugins need to be fixed...
            There are two ways to suspend user accounts:
            1/ user.auth==nologin
            2/ user.suspended==1

            All messaging plugins need to be fixed, at the same time we could prevent messaging to deleted users too.
            Petr Skoda made changes -
            Assignee Andrew Davis [ andyjdavis ] Petr Škoda (skodak) [ skodak ]
            Petr Skoda made changes -
            Status Open [ 1 ] Development in progress [ 3 ]
            Petr Skoda made changes -
            Description There are two ways to suspend user accounts:
            1/ user.auth==nologin
            2/ user.suspended==1

            All messaging plugins need to be fixed, at the same time we could prevent messaging to deleted users too.
            There are two ways to suspend user accounts:
            1/ user.auth==nologin
            2/ user.suspended==1

            All messaging plugins need to be fixed, at the same time we could prevent messaging to deleted users too.

            OH! Please note that word "messaging" here does not mean the direct sending of messages between users in moodle, it is fine to send messages to users this way, they just do not get notification via email/jabber/etc.
            Hide
            Petr Skoda added a comment -

            The only potential problem here is performance - the messaging API could be refetching incomplete user records, but I suppose we ignored this till now and it should be probably adressed in 2.3 together with int parameters to message_send()

            To integrators: please cherry pick to 2.1.x and 2.2.x

            Show
            Petr Skoda added a comment - The only potential problem here is performance - the messaging API could be refetching incomplete user records, but I suppose we ignored this till now and it should be probably adressed in 2.3 together with int parameters to message_send() To integrators: please cherry pick to 2.1.x and 2.2.x
            Petr Skoda made changes -
            Status Development in progress [ 3 ] Waiting for integration review [ 10010 ]
            Pull Master Diff URL https://github.com/skodak/moodle/compare/master...w52_MDL-30948_m23_messagesuspended
            Pull Master Branch w52_MDL-30948_m23_messagesuspended
            Pull from Repository git://github.com/skodak/moodle.git
            Fix Version/s 2.1.4 [ 11452 ]
            Fix Version/s 2.2.1 [ 11456 ]
            Fix Version/s 2.3 [ 10657 ]
            Testing Instructions 1/ suspend user account (see MDL-30548 for new admin UI)
            2/ try to message suspended user
            Petr Skoda made changes -
            Link This issue has been marked as being related by MDL-30548 [ MDL-30548 ]
            Eloy Lafuente (stronk7) made changes -
            Currently in integration Yes [ 10041 ]
            Eloy Lafuente (stronk7) made changes -
            Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
            Integrator stronk7
            Petr Skoda made changes -
            Parent MDL-28351 [ 45778 ]
            Issue Type Bug [ 1 ] Sub-task [ 5 ]
            Hide
            Petr Skoda added a comment -

            I have added one more commit preventing generation of new passwords in cron for suspended accounts - because they would can not get the email

            Show
            Petr Skoda added a comment - I have added one more commit preventing generation of new passwords in cron for suspended accounts - because they would can not get the email
            Petr Skoda made changes -
            Priority Minor [ 4 ] Major [ 3 ]
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Do we really need to fetch the whole user record? At least reducing it to some sort of MESSAGING_REQUIRED_FIELDS or any other collection of fields would be better.

            Anyway, surely something to improve in 2.3... so integrating... ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Do we really need to fetch the whole user record? At least reducing it to some sort of MESSAGING_REQUIRED_FIELDS or any other collection of fields would be better. Anyway, surely something to improve in 2.3... so integrating... ciao
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Integrated (21, 22 and master), thanks!

            Note for 21 I've made a new commit because the whole cronlib order was revamped for 22 and yours did not apply clean.

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated (21, 22 and master), thanks! Note for 21 I've made a new commit because the whole cronlib order was revamped for 22 and yours did not apply clean.
            Eloy Lafuente (stronk7) made changes -
            Status Integration review in progress [ 10004 ] Waiting for testing [ 10005 ]
            Affects Version/s 2.3 [ 10657 ]
            Fix Version/s 2.3 [ 10657 ]
            Hide
            Petr Skoda added a comment -

            grrr, sorry and thanks, my memory is getting really bad

            Show
            Petr Skoda added a comment - grrr, sorry and thanks, my memory is getting really bad
            moodle.com made changes -
            Tester phalacee
            Jason Fowler made changes -
            Status Waiting for testing [ 10005 ] Testing in progress [ 10011 ]
            Hide
            Jason Fowler added a comment -

            Took me a while to realize this was referring to the emails that go with the messages, and not the messages themselves ... but once I knew what I was doing the testing passed nicely

            Show
            Jason Fowler added a comment - Took me a while to realize this was referring to the emails that go with the messages, and not the messages themselves ... but once I knew what I was doing the testing passed nicely
            Jason Fowler made changes -
            Status Testing in progress [ 10011 ] Tested [ 10006 ]
            Michael de Raadt made changes -
            Labels triaged
            Hide
            Eloy Lafuente (stronk7) added a comment -

            The git and cvs repositories are happy receiving your very first contribution to Moodle for 2012. Happy new year!

            Closing, ciao

            Show
            Eloy Lafuente (stronk7) added a comment - The git and cvs repositories are happy receiving your very first contribution to Moodle for 2012. Happy new year! Closing, ciao
            Eloy Lafuente (stronk7) made changes -
            Status Tested [ 10006 ] Closed [ 6 ]
            Resolution Fixed [ 1 ]
            Currently in integration Yes [ 10041 ]
            Integration date 05/Jan/12
            Hide
            Helen Foster added a comment -

            Any chance we can get rid of the no login auth method now we have the 'Suspended account' setting?

            Show
            Helen Foster added a comment - Any chance we can get rid of the no login auth method now we have the 'Suspended account' setting?

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: