5.2.0-git
04/24/2014

[#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 06/25/2009 (1764 days ago)
Due
Updated 01/24/2013 (455 days ago)
Assigned 06/28/2009 (1761 days ago)
Resolved 01/24/2013 (455 days ago)
Milestone
Patch Yes

History
01/24/2013 01:08:06 AM Michael Slusarz Comment #3
State ⇒ Resolved
Taken from Horde DevelopersHorde Developers
Assigned to Michael Slusarz
Reply to this comment
Ingo 3.0.3.
01/24/2013 01:07:57 AM 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(-)

http://git.horde.org/horde-git/-/commit/faf21c14a3a43cfb556320bbcb15cc0ba9182156
06/28/2009 04:22:38 PM Chuck Hagenbuch State ⇒ Assigned
Assigned to Horde DevelopersHorde Developers
 
06/25/2009 08:11:00 AM 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.