Opened 16 years ago

Closed 16 years ago

Last modified 13 years ago

#8716 closed (fixed)

When a authenticated user post a comments, the field's form is not populate with data of user

Reported by: jmad Owned by: jmad
Component: contrib.comments Version: dev
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

When a authenticated user post a comments, the field's form is not populate with full name and email of the user. This bug stems from the fact that the code is testing if "name" and "email" are present or not in "Data". If they are not present, then the code uses data from the user. Ors these two fields are always present in "data". It must test if they are empty or not and not if they are present.

This bug is in svn and beta 2 version.

Attachments (1)

patch_comments.diff (725 bytes ) - added by jmad 16 years ago.

Download all attachments as: .zip

Change History (6)

by jmad, 16 years ago

Attachment: patch_comments.diff added

comment:1 by jmad, 16 years ago

Owner: changed from nobody to jmad
Status: newassigned

comment:2 by anonymous, 16 years ago

milestone: 1.0

comment:3 by Malcolm Tredinnick, 16 years ago

Triage Stage: UnreviewedAccepted

The standard way to test if a dictionary key has non-empty content is if data["name"]: .... The patch style looks more like C and makes the reader wonder why it's trying to do something special.

comment:4 by Jacob, 16 years ago

Resolution: fixed
Status: assignedclosed

(In [8751]) Fixed #8716: correctly handle name and email in comments from authenticated users.

comment:5 by Jacob, 13 years ago

milestone: 1.0

Milestone 1.0 deleted

Note: See TracTickets for help on using tickets.
Back to Top