Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#14791 closed (fixed)

Clarify additional note on save()

Reported by: OldTroll Owned by: nobody
Component: Documentation Version: 1.2
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

I found the save() if form.error phrasing to be confusing. Admittedly I'm reviewing the documentation on a kindle so the display is a little off, but I think specifying that it must evaluate to a True value is less ambiguous than a small snippet of code without any contextual information.

Attachments (1)

clarify_save.diff​ (589 bytes ) - added by OldTroll 13 years ago.
Clarify the text about the save() method

Download all attachments as: .zip

Change History (3)

by OldTroll, 13 years ago

Attachment: clarify_save.diff​ added

Clarify the text about the save() method

comment:1 by Tim Graham, 13 years ago

Resolution: → fixed
Status: new → closed

(In [14740]) Fixed #14791 - Clarify additional note on save(). Thanks OldTroll.

comment:2 by Tim Graham, 13 years ago

(In [14741]) [1.2.X] Fixed #14791 - Clarify additional note on save(). Thanks OldTroll.

Backport of r14740 from trunk.

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