Merge lp:~openerp-dev/openerp-web/7.0-imp-description-fka into lp:openerp-web/7.0

[Foram Katharotiya (OpenERP)]

Hello,

 - Fix the issue of False is returned when saving data in discription field even it's stay in empty.
   I have improved code as per your suggestion.

Thanks,
FKA.

[Xavier (Open ERP) (xmo-deactivatedaccount)]

What was a generic problem? Could you explain the issue this intends to fix?

review: Needs Information

[Fabien (Open ERP) (fp-tinyerp)]

It looks like it's related to this task:
  https://openerp.my.openerp.com/#id=5201&view_type=form&model=project.task

[Xavier (Open ERP) (xmo-deactivatedaccount)]

Looks like it.

This proposal completely removes all display of all text fields in "readonly" mode. Even when there is content in the field.

review: Disapprove

[lp:~openerp-dev/openerp-web/7.0-imp-description-fka]

3829. By Foram Katharotiya (OpenERP) on 2013-03-13

[MERGE] with 7.0

3830. By Foram Katharotiya (OpenERP) on 2013-03-13

[IMP] improve code

[Xavier (Open ERP) (xmo-deactivatedaccount)]

Seems to have been fixed separately in <email address hidden>

Unmerged revisions

3830. By Foram Katharotiya (OpenERP) on 2013-03-13

[IMP] improve code

3829. By Foram Katharotiya (OpenERP) on 2013-03-13

[MERGE] with 7.0

3828. By Foram Katharotiya (OpenERP) on 2013-03-13

[IMP] fix the issue of showing False value in discription

3827. By Foram Katharotiya (OpenERP) on 2013-03-12

[MERGE] with 7.0

3826. By Foram Katharotiya (OpenERP) on 2013-03-08

[IMP] set empty description fields

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2013-03-12 17:05:15 +0000
+++ addons/web/static/src/js/view_form.js	2013-03-13 06:48:20 +0000
@@ -2602,8 +2602,7 @@
                 this.$textarea.trigger("autosize");
             }
         } else {
-            var txt = this.get("value");
-            this.$(".oe_form_text_content").text(txt);
+            this.$(".oe_form_text_content").text(this.get("value") || '');
         }
     },
     is_syntax_valid: function() {