Attachment 31788 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.05 KB, created by Marc Véron on 2014-09-20 08:51:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2014-09-20 08:51:35 UTC

Size: 2.05 KB

patch

obsolete

>From 8f6a90aa52999cf0c4236389fd733a00507df0f1 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 as 'required' and it is not longer possible to store a news without having at least a title.
>
>To test:
>Got to Tools->News
>Create news or edit existing news
>Field "Title" 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 |    7 ++++---
> 1 file changed, 4 insertions(+), 3 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..e1159f8 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>
>@@ -141,7 +141,8 @@ Edit news item[% ELSE %]Add news item[% END %][% ELSE %]News[% END %]</div>
>                 [% END %]
>             </li>
>             <li><label for="new">News: </label>
>-            <textarea name="new" id="new"  cols="75" rows="10">[% new_detail.new %]</textarea></li>
>+            <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