[silva.app.news][Sylvain Viollon] Fix broken widgets (they didn'...

[email protected]
Newsgroups gmane.comp.web.zope.silva.cvs
Message-ID <[email protected]>
author:    Sylvain Viollon
date:      Fri Aug 23 13:38:55 2013 +0200
revision:  2035:20edea9d98f2 in silva.app.news
branch:    2.4
details:   https://hg.infrae.com/silva.app.news?cmd=changeset;node=20edea9d98f2
modified:  src/silva/app/news/widgets/static/recurrence.js src/silva/app/news/widgets/static/tree.js src/silva/app/news/widgets/tree.py
added:     
removed:   
log:       Fix broken widgets (they didn't load).


diffstat:

 src/silva/app/news/widgets/static/recurrence.js |  9 +--------
 src/silva/app/news/widgets/static/tree.js       |  9 ++-------
 src/silva/app/news/widgets/tree.py              |  7 ++++---
 3 files changed, 7 insertions(+), 18 deletions(-)

diffs (82 lines):

diff -r ee7e4a0d0601 -r 20edea9d98f2 src/silva/app/news/widgets/static/recurrence.js
--- a/src/silva/app/news/widgets/static/recurrence.js	Wed Jul 31 17:05:33 2013 +0200
+++ b/src/silva/app/news/widgets/static/recurrence.js	Fri Aug 23 13:38:55 2013 +0200
@@ -1039,8 +1039,7 @@
         infrae.ui.ShowDialog($popup);
     });
 
-
-    var create_recurrence = function () {
+    $(document).on('loadwidget-smiform', '.form-fields-container', function(event, data) {
         $(this).find('.recurrence-widget').each(function() {
             var $widget = $(this);
             var $input = $widget.find('input.recurrence-data');
@@ -1049,13 +1048,7 @@
                 humanize(parse_recurrence_data(value),
                          humanize_translations));
         });
-    };
-
-    $('.form-fields-container').live('loadwidget-smiform', function(event) {
-        $(this).invoke(create_recurrence);
         event.stopPropagation();
     });
 
-    $(document).ready(create_recurrence);
-
 })(jQuery, infrae);
diff -r ee7e4a0d0601 -r 20edea9d98f2 src/silva/app/news/widgets/static/tree.js
--- a/src/silva/app/news/widgets/static/tree.js	Wed Jul 31 17:05:33 2013 +0200
+++ b/src/silva/app/news/widgets/static/tree.js	Fri Aug 23 13:38:55 2013 +0200
@@ -1,6 +1,6 @@
 (function ($) {
 
-    function create_trees() {
+    $(document).on('loadwidget-smiform', '.form-fields-container', function(event, data) {
         var $trees = $(this).find(".field-tree-widget");
 
         $trees.each(function () {
@@ -21,7 +21,7 @@
             });
 
             if (!readonly) {
-                $tree.delegate('a', 'click', function() {
+                $tree.on('click', 'a', function() {
                     var values = [];
 
                     $.each($tree.jstree('get_checked'), function() {
@@ -32,13 +32,8 @@
                 });
             };
         });
-    };
-
-    $('.form-fields-container').live('loadwidget-smiform', function(event) {
-        $(this).invoke(create_trees);
         event.stopPropagation();
     });
 
-    $(document).ready(create_trees);
 
 })(jQuery);
diff -r ee7e4a0d0601 -r 20edea9d98f2 src/silva/app/news/widgets/tree.py
--- a/src/silva/app/news/widgets/tree.py	Wed Jul 31 17:05:33 2013 +0200
+++ b/src/silva/app/news/widgets/tree.py	Fri Aug 23 13:38:55 2013 +0200
@@ -125,9 +125,7 @@
     grok.adapts(TreeSchemaField, Interface, Interface)
     grok.name(str(INPUT))
 
-    @property
-    def selected_only(self):
-        return grok.name.bind().get(self) == str(DISPLAY)
+    selected_only = False
 
     def update(self):
         need(ITreeResources)
@@ -169,3 +167,6 @@
 
 class TreeWidgetDisplay(TreeWidgetInput):
     grok.name(str(DISPLAY))
+
+    selected_only = True
+
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.