From bb97a0eedeb237332869645077e3a06a9236b758 Mon Sep 17 00:00:00 2001
From: James <james@sourcefabric-DX4840.(none)>
Date: Fri, 19 Aug 2011 13:09:10 -0400
Subject: [PATCH 1/3] CC-2704: Support Setting UI tweak

- move station name under publicise
- make it required field
---
 .../application/forms/SupportSettings.php     |  3 +-
 .../views/scripts/form/support-setting.phtml  | 31 ++++++++++---------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/airtime_mvc/application/forms/SupportSettings.php b/airtime_mvc/application/forms/SupportSettings.php
index 12bb20237..ef6b9f2fa 100644
--- a/airtime_mvc/application/forms/SupportSettings.php
+++ b/airtime_mvc/application/forms/SupportSettings.php
@@ -16,8 +16,9 @@ class Application_Form_SupportSettings extends Zend_Form
         $this->addElement('text', 'stationName', array(
             'class'      => 'input_text',
             'label'      => 'Station Name',
-            'required'   => false,
+            'required'   => true,
             'filters'    => array('StringTrim'),
+            'validator'  => array('NotEmpty'),
             'value' => Application_Model_Preference::GetValue("station_name"),
             'decorators' => array(
                 'ViewHelper'
diff --git a/airtime_mvc/application/views/scripts/form/support-setting.phtml b/airtime_mvc/application/views/scripts/form/support-setting.phtml
index 014ef23cc..bb24b89ed 100644
--- a/airtime_mvc/application/views/scripts/form/support-setting.phtml
+++ b/airtime_mvc/application/views/scripts/form/support-setting.phtml
@@ -1,19 +1,5 @@
 <fieldset class="padded">
     <dl class="zend_form">
-        <dt id="stationName-label" class="block-display">
-            <label class="required" for="stationName"><?php echo $this->element->getElement('stationName')->getLabel() ?> :
-            </label>
-        </dt>
-        <dd id="stationName-element" class="block-display">
-            <?php echo $this->element->getElement('stationName') ?>
-            <?php if($this->element->getElement('stationName')->hasErrors()) : ?>
-                <ul class='errors'>
-                    <?php foreach($this->element->getElement('stationName')->getMessages() as $error): ?>
-                        <li><?php echo $error; ?></li>
-                    <?php endforeach; ?>
-                </ul>
-            <?php endif; ?>
-        </dd>
         <dd id="SupportFeedback-element" style="width:90%;">
             <div class="info-text">Help Airtime improve by letting us know you are using it. This info
             will be collected regularly in order to improve your user experience.</div>
@@ -47,7 +33,22 @@
         </dd>
     </dl>
         <div class="info-text" style="clear: both;padding: 4px 0 4px 15px;"><p> Note: In order to promote your station, "Send support feedback" must be enabled</p></div>
-    <dl id="public-info" style="display:none;"> 
+    <dl id="public-info" style="display:none;">
+        <dt id="stationName-label" class="block-display">
+            <label class="required" for="stationName"><?php echo $this->element->getElement('stationName')->getLabel() ?>
+                <span class="info-text-small">(Required)</span>:
+            </label>
+        </dt>
+        <dd id="stationName-element" class="block-display">
+            <?php echo $this->element->getElement('stationName') ?>
+            <?php if($this->element->getElement('stationName')->hasErrors()) : ?>
+                <ul class='errors'>
+                    <?php foreach($this->element->getElement('stationName')->getMessages() as $error): ?>
+                        <li><?php echo $error; ?></li>
+                    <?php endforeach; ?>
+                </ul>
+            <?php endif; ?>
+        </dd>
         <dt id="Phone-label" class="block-display">
             <label class="optional" for="Phone"><?php echo $this->element->getElement('Phone')->getLabel() ?></label>
             <span class="info-text-small">(for verification purposes only, will not be published)</span>

From 8e108e859678fe607ce885bfff8a886b6206d83e Mon Sep 17 00:00:00 2001
From: James <james@sourcefabric-DX4840.(none)>
Date: Fri, 19 Aug 2011 14:58:31 -0400
Subject: [PATCH 2/3] CC-2704: Change text

- fixed
---
 .../application/views/scripts/preference/support-setting.phtml  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/airtime_mvc/application/views/scripts/preference/support-setting.phtml b/airtime_mvc/application/views/scripts/preference/support-setting.phtml
index 85176345b..1bc60e81e 100644
--- a/airtime_mvc/application/views/scripts/preference/support-setting.phtml
+++ b/airtime_mvc/application/views/scripts/preference/support-setting.phtml
@@ -1,5 +1,5 @@
 <div class="ui-widget ui-widget-content block-shadow simple-formblock clearfix padded-strong preferences">
-<h2>Support Setting</h2>
+<h2>Support Settings</h2>
     <form method="post" action="/Preference/support-setting" enctype="multipart/form-data">
         <div id="support-settings">
             <?php echo $this->statusMsg ?>

From f2f599f7232a70c465b7b4e80d07ec3898729ab9 Mon Sep 17 00:00:00 2001
From: James <james@sourcefabric-DX4840.(none)>
Date: Fri, 19 Aug 2011 15:00:45 -0400
Subject: [PATCH 3/3] CC-2704: String change

-fixed
---
 .../application/views/scripts/preference/stream-setting.phtml   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/airtime_mvc/application/views/scripts/preference/stream-setting.phtml b/airtime_mvc/application/views/scripts/preference/stream-setting.phtml
index f28215968..e34d6b1f4 100644
--- a/airtime_mvc/application/views/scripts/preference/stream-setting.phtml
+++ b/airtime_mvc/application/views/scripts/preference/stream-setting.phtml
@@ -1,5 +1,5 @@
 <div class="ui-widget ui-widget-content block-shadow simple-formblock clearfix padded-strong stream-config">
-    <h2>Stream configuration</h2>
+    <h2>Stream Settings</h2>
 <?php echo $this->statusMsg;?>
     <form method="post" action="/Preference/stream-setting" enctype="application/x-www-form-urlencoded">
     <fieldset class="padded">