Code

Opened 3 years ago

Closed 3 years ago

Last modified 3 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:
Needs tests: Patch needs improvement:
Easy pickings: UI/UX:

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 3 years ago.
Clarify the text about the save() method

Download all attachments as: .zip

Change History (3)

Changed 3 years ago by OldTroll

Clarify the text about the save() method

comment:1 Changed 3 years ago by timo

  • Resolution set to fixed
  • Status changed from new to closed

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

comment:2 Changed 3 years ago by timo

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

Backport of r14740 from trunk.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.