API page: http://api.drupal.org/api/drupal/modules--user--user.module/function/_us...

Describe the problem you have found:

The list of possible values for the argument $op is really hardly readable, please make a

  • list out of it.
    Thank you!
  • Files: 
    CommentFileSizeAuthor
    #12 user-indentation-1416212-12.patch1.97 KBpontus_nilsson
    FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch user-indentation-1416212-12.patch. Unable to apply patch. See the log in the details link for more information.
    [ View ]
    #10 user-indentation-1416212-9.patch1.97 KBpontus_nilsson
    PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
    [ View ]
    #6 user-indentation-1416212-6.patch2.12 KBpontus_nilsson
    PASSED: [[SimpleTest]]: [MySQL] 33,334 pass(es).
    [ View ]
    #4 user-indentation-1416212-4.patch2.1 KBpontus_nilsson
    PASSED: [[SimpleTest]]: [MySQL] 33,353 pass(es).
    [ View ]
    #2 user-indentation-1416212-2.patch2.09 KBpontus_nilsson
    PASSED: [[SimpleTest]]: [MySQL] 33,354 pass(es).
    [ View ]

    Comments

    Title:Documentation problem with _user_mail_notify_user_mail_notify -- options for $op need to be a list
    Version:6.x-dev» 8.x-dev
    Issue tags:+needs backport to D6, +Novice, +needs backport to D7

    Thanks for reporting this! Good novice project, and needs to be done in Drupal 8, then 7, then 6.

    Status:Active» Needs review
    StatusFileSize
    new2.09 KB
    PASSED: [[SimpleTest]]: [MySQL] 33,354 pass(es).
    [ View ]

    Patch adds correct indentation and "-" for the possible values for function _user_mail_notify() description. Patch is against 8.x

    Status:Needs review» Needs work

    Looks close, thanks!

    Please check out our list guidelines though -- every list item should end with a . :
    http://drupal.org/node/1354#lists
    You might also verify that none of those lines exceeds 80 characters, just in case.

    Also, we need a blank line between @param and @return.

    Thanks for catching and fixing the indentation on the param/returns too!

    StatusFileSize
    new2.1 KB
    PASSED: [[SimpleTest]]: [MySQL] 33,353 pass(es).
    [ View ]

    Thank you for your guideance.
    New patch adds "." to end of list items and a blank line between @param and @return.
    Regarding line lengths, does this count as 80 or 81?

    *   - 'register_no_approval_required': Welcome message when user self-registers.

    Status:Needs work» Needs review

    StatusFileSize
    new2.12 KB
    PASSED: [[SimpleTest]]: [MySQL] 33,334 pass(es).
    [ View ]

    New patch ensures line length of 80 characters.

    Status:Needs review» Reviewed & tested by the community

    Looks good, thanks!

    Status:Reviewed & tested by the community» Fixed

    Looks great!

    Committed and pushed to 8.x and 7.x. Thanks!

    Version:8.x-dev» 6.x-dev
    Status:Fixed» Patch (to be ported)

    Still needs backport to D6.

    Status:Patch (to be ported)» Needs review
    StatusFileSize
    new1.97 KB
    PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
    [ View ]

    Attached patch is against 6.x branch. Changed the indentation, added "-" to seperate list items, added punctuation and blank row before return value.

    Status:Needs review» Needs work

    Thanks! It looks like there is one other mistake to correct in 6.x -- there are two spaces instead of one after the . in this line:

    + *   The user object of the account being notified.  Must contain at

    Status:Needs work» Needs review
    StatusFileSize
    new1.97 KB
    FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch user-indentation-1416212-12.patch. Unable to apply patch. See the log in the details link for more information.
    [ View ]

    New patch fixes the two spaces noticed in #11.

    Status:Needs review» Reviewed & tested by the community

    Looks good, thanks!

    Status:Reviewed & tested by the community» Fixed

    Thanks a lot, committed, pushed.

    Automatically closed -- issue fixed for 2 weeks with no activity.