[#8376] vacation allows empty subject/reason
Summary vacation allows empty subject/reason
Queue Ingo
Queue Version 1.2.1
Type Enhancement
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester crohmann (at) netcologne (dot) de
Created 2009-06-25 (2591 days ago)
Updated 2013-01-24 (1282 days ago)
Assigned 2009-06-28 (2588 days ago)
Resolved 2013-01-24 (1282 days ago)
Patch Yes

2013-01-24 01:08:06 Michael Slusarz Comment #3
State ⇒ Resolved
Taken from Horde DevelopersHorde Developers
Assigned to Michael Slusarz
Reply to this comment
Ingo 3.0.3.
2013-01-24 01:07:57 Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit faf21c14a3a43cfb556320bbcb15cc0ba9182156
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Wed Jan 23 18:06:28 2013 -0700

     [mms] Require non-empty subject and reason fields when creating a 
vacation message (crohmann@netcologne.de; Request #8376).

  ingo/docs/CHANGES          |    2 ++
  ingo/lib/Form/Vacation.php |    4 ++--
  ingo/package.xml           |    2 ++
  3 files changed, 6 insertions(+), 2 deletions(-)

2009-06-28 16:22:38 Chuck Hagenbuch State ⇒ Assigned
Assigned to Horde DevelopersHorde Developers
2009-06-25 08:11:00 crohmann (at) netcologne (dot) de Comment #1
State ⇒ New
New Attachment: vacation_must_subject_reason.diff Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Ingo
Summary ⇒ vacation allows empty subject/reason
Type ⇒ Enhancement
Priority ⇒ 1. Low
Reply to this comment
A user can just click on vacation and then press activate without even 
entering ANY information. This leads to unwanted mail junk without the 
user even noticing.

The attached patch sets the fields "subject" and "reason" as MUST enter.

One might argue that now there always has to be text in those fields 
even though the rule is not enabled. Otherwise there had to be more 
code to check for things, this patch does not change much.