Attachment 31579 Details for Bug 12740 – Bug 12740 - It should not be possible to store empty news

[patch] Bug 12740 - It should not be possible to store empty news

Bug-12740---It-should-not-be-possible-to-store-emp.patch (text/plain), 2.42 KB, created by Marc Véron on 2014-09-13 16:21:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2014-09-13 16:21:08 UTC

Size: 2.42 KB

patch

obsolete

>From 383758e954f05e608b76a65045576f3619c285fe Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch>
>Date: Sat, 13 Sep 2014 18:10:06 +0200
>Subject: [PATCH] Bug 12740 - It should not be possible to store empty news
>
>This patch marks the title field and the textarea field as 'required' and it is
>not longer possible to store a news without having at least a title.
>
>Note: The 'required' validation for a textarea does not work properly, that's why
>the textarea for the news text is not validated, only marked as 'required'
>
>To test:
>Got to Tools->News
>Create news or edit existing news
>Fields "Title" and "News" should have a red label and should be marked as "required"
>It should not be possible to save with empty field "Title"
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt |    9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt
>index 552fb6e..0b33ee3 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt
>@@ -119,8 +119,8 @@ Edit news item[% ELSE %]Add news item[% END %][% ELSE %]News[% END %]</div>
>                 </select>
>             </li>
>             <li>
>-                <label for="title">Title: </label>
>-                <input id="title" size="30" type="text" name="title" value="[% new_detail.title %]" />
>+                <label for="title" class="required">Title: </label>
>+                <input id="title" size="30" type="text" name="title" value="[% new_detail.title %]" required="required" class="required" /> <span class="required">Required</span>
>             </li>
>             <li>
>                 <label for="from">Publication date: </label>
>@@ -140,8 +140,9 @@ Edit news item[% ELSE %]Add news item[% END %][% ELSE %]News[% END %]</div>
>                     <input id="number" size="3" name="number" type="text" />
>                 [% END %]
>             </li>
>-            <li><label for="new">News: </label>
>-            <textarea name="new" id="new"  cols="75" rows="10">[% new_detail.new %]</textarea></li>
>+            <li><label for="new" class="required">News: </label> <span class="required">Required</span>
>+            <textarea name="new" id="new"  cols="75" rows="10">[% new_detail.new %]</textarea>
>+            </li>
>             </ol>
> 			</fieldset>
>   
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 12740: 30647 | 31579 | 31788 | 31789 | 31790 | 31791 | 31804 | 31805 | 31806