Code cleanup

bzr revid: ddm@openerp.com-20130827082232-4irtyjd4t67pz5hl

diff --git a/addons/website/static/src/js/website.js b/addons/website/static/src/js/website.js
index 4637e28..41dc7e0 100644 (file)
--- a/addons/website/static/src/js/website.js
+++ b/addons/website/static/src/js/website.js
@@ -235,7 +235,7 @@
     website.seo.Configurator = openerp.Widget.extend({
         template: 'website.seo_configuration',
         events: {
-            'click button[data-action=add]': 'add',
+            'click button[data-action=add]': 'addKeyword',
             'click a[data-action=update]': 'update',
             'hidden': 'close'
         },
@@ -247,15 +247,12 @@
 
             this.$el.modal();
         },
-        addKeyword: function (word) {
+        addKeyword: function () {
+            var word = this.$el.find('input[name=seo_page_keywords]').val();
             new website.seo.Keyword({
                 keyword: word
             }).appendTo(this.$el.find('.js_seo_keywords_list'));
         },
-        add: function () {
-            var word = this.$el.find('input[name=seo_page_keywords]').val();
-            this.addKeyword(word);
-        },
         update: function () {
             // TODO: Persist changes
         },

diff --git a/addons/website/static/src/xml/website.xml b/addons/website/static/src/xml/website.xml
index 9b7c224..978c1b2 100644 (file)
--- a/addons/website/static/src/xml/website.xml
+++ b/addons/website/static/src/xml/website.xml
@@ -64,13 +64,13 @@
                 <div class="control-group">
                     <label class="control-label" for="seo_page_url">URL</label>
                     <div class="controls">
-                        <input type="text" name="seo_page_url" required="True" size="70" readonly="readonly"/>
+                        <input type="text" name="seo_page_url" size="70" readonly="readonly"/>
                     </div>
                 </div>
                 <div class="control-group">
                     <label class="control-label" for="seo_page_title">Title</label>
                     <div class="controls">
-                        <input type="text" name="seo_page_title" required="True" size="70"/>
+                        <input type="text" name="seo_page_title" size="70"/>
                     </div>
                 </div>
                 <div class="control-group">
@@ -83,7 +83,7 @@
                     <label class="control-label" for="seo_page_keywords">Keywords</label>
                     <div class="controls">
                         <div class="input-append">
-                            <input type="text" name="seo_page_keywords" required="True" size="30" placeholder="New keyword"/>
+                            <input type="text" name="seo_page_keywords" size="30" placeholder="New keyword"/>
                             <button data-action="add" class="btn" type="button">Add</button>
                         </div>
                         <ul class="js_seo_keywords_list">