Changeset 19:48bd45f41903 in ralphm-patches for pubsub-create-configure.patch


Ignore:
Timestamp:
Dec 29, 2009, 3:35:35 PM (11 years ago)
Author:
Ralph Meijer <ralphm@…>
Branch:
default
Message:

Further adjustments on type checking, a first test for parsing subscription
options.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • pubsub-create-configure.patch

    r14 r19  
    1 diff -r 53bd69434e51 wokkel/data_form.py
    2 --- a/wokkel/data_form.py       Mon Apr 27 17:53:05 2009 +0200
    3 +++ b/wokkel/data_form.py       Fri May 01 08:34:36 2009 +0200
    4 @@ -251,10 +251,10 @@
     1diff -r 9a749c388cda wokkel/data_form.py
     2--- a/wokkel/data_form.py       Tue Dec 29 14:55:13 2009 +0100
     3+++ b/wokkel/data_form.py       Tue Dec 29 15:06:54 2009 +0100
     4@@ -253,10 +253,10 @@
    55             field['var'] = self.var
    66 
     
    1414+                value = unicode(value)
    1515 
    16              field.addElement('value', content=value)
    17  
    18 @@ -489,3 +489,25 @@
    19              values[name] = value
    20  
    21          return values
     16             field.addElement('value', content=unicode(value))
     17 
     18@@ -564,3 +564,25 @@
     19             else:
     20                 # Unknown field, ignoring
     21                 pass
    2222+
    2323+
     
    4242+
    4343+    return None
    44 diff -r 53bd69434e51 wokkel/pubsub.py
    45 --- a/wokkel/pubsub.py  Mon Apr 27 17:53:05 2009 +0200
    46 +++ b/wokkel/pubsub.py  Fri May 01 08:34:36 2009 +0200
    47 @@ -233,7 +233,7 @@
     44diff -r 9a749c388cda wokkel/pubsub.py
     45--- a/wokkel/pubsub.py  Tue Dec 29 14:55:13 2009 +0100
     46+++ b/wokkel/pubsub.py  Tue Dec 29 15:06:54 2009 +0100
     47@@ -234,7 +234,7 @@
    4848         'optionsSet': ['nodeOrEmpty', 'jid', 'options'],
    4949         'subscriptions': [],
     
    5454         'configureGet': ['nodeOrEmpty'],
    5555         'configureSet': ['nodeOrEmpty', 'configure'],
    56 @@ -251,30 +251,6 @@
     56@@ -252,30 +252,6 @@
    5757         self.verb = verb
    5858 
     
    8585         """
    8686         Parse the required node identifier out of the verbElement.
    87 @@ -365,7 +341,7 @@
     87@@ -366,7 +342,7 @@
    8888         """
    8989         Parse node type out of a request for the default node configuration.
     
    9494             values = form.getValues()
    9595             self.nodeType = values.get('pubsub#node_type', 'leaf')
    96 @@ -377,7 +353,7 @@
     96@@ -378,7 +354,7 @@
    9797         """
    9898         Parse options out of a request for setting the node configuration.
     
    103103             if form.formType == 'submit':
    104104                 self.options = form.getValues()
    105 @@ -389,6 +365,46 @@
     105@@ -390,6 +366,46 @@
    106106             raise BadRequest(text="Missing configuration form")
    107107 
     
    150150     def _parse_itemIdentifiers(self, verbElement):
    151151         """
    152 @@ -436,7 +452,7 @@
     152@@ -437,7 +453,7 @@
    153153 
    154154 
     
    159159             if form.formType == 'submit':
    160160                 self.options = form.getValues()
    161 @@ -447,6 +463,7 @@
    162          else:
    163              raise BadRequest(text="Missing options form")
     161@@ -468,6 +484,7 @@
     162             self._render_options(optionsElement)
     163 
    164164 
    165165+
     
    167167         """
    168168         Parse the publish-subscribe verb and parameters out of a request.
    169 @@ -630,7 +647,8 @@
     169@@ -663,7 +680,8 @@
    170170         pass
    171171 
     
    177177         Create a publish subscribe node.
    178178 
    179 @@ -638,11 +656,14 @@
     179@@ -671,11 +689,14 @@
    180180         @type service: L{JID}
    181181         @param nodeIdentifier: Optional suggestion for the id of the node.
     
    192192         def cb(iq):
    193193             try:
    194 @@ -915,8 +936,6 @@
     194@@ -952,8 +973,6 @@
    195195             d = self.getNodes(requestor, target)
    196196         else:
     
    201201         d.addCallback(lambda nodes: [disco.DiscoItem(target, node)
    202202                                      for node in nodes])
    203 @@ -1062,6 +1081,13 @@
     203@@ -1081,6 +1100,13 @@
    204204         return processedValues
    205205 
     
    215215         if request.nodeType not in ('leaf', 'collection'):
    216216             raise error.StanzaError('not-acceptable')
    217 diff -r 53bd69434e51 wokkel/test/test_pubsub.py
    218 --- a/wokkel/test/test_pubsub.py        Mon Apr 27 17:53:05 2009 +0200
    219 +++ b/wokkel/test/test_pubsub.py        Fri May 01 08:34:36 2009 +0200
     217diff -r 9a749c388cda wokkel/test/test_pubsub.py
     218--- a/wokkel/test/test_pubsub.py        Tue Dec 29 14:55:13 2009 +0100
     219+++ b/wokkel/test/test_pubsub.py        Tue Dec 29 15:06:54 2009 +0100
    220220@@ -19,7 +19,7 @@
    221221 from wokkel.test.helpers import TestableRequestHandlerMixin, XmlStreamStub
     
    268268         """
    269269         Test sending delete request.
    270 @@ -935,6 +969,7 @@
     270@@ -966,6 +1000,7 @@
    271271         self.assertEqual(JID('user@example.org'), request.sender)
    272272         self.assertEqual(JID('pubsub.example.org'), request.recipient)
     
    276276 
    277277     def test_fromElementCreateInstant(self):
    278 @@ -955,6 +990,77 @@
     278@@ -986,6 +1021,77 @@
    279279         self.assertIdentical(None, request.nodeIdentifier)
    280280 
     
    354354         """
    355355         Test parsing a request for the default node configuration.
    356 @@ -1740,6 +1846,51 @@
     356@@ -1771,6 +1877,51 @@
    357357         return d
    358358 
     
    406406         """
    407407         A default request should result in
    408 @@ -1773,7 +1924,7 @@
     408@@ -1804,7 +1955,7 @@
    409409             self.assertEqual(NS_PUBSUB_OWNER, element.uri)
    410410             self.assertEqual(NS_PUBSUB_OWNER, element.default.uri)
     
    415415         self.resource.getConfigurationOptions = getConfigurationOptions
    416416         self.resource.default = default
    417 @@ -1902,7 +2053,7 @@
     417@@ -1933,7 +2084,7 @@
    418418             self.assertEqual(NS_PUBSUB_OWNER, element.uri)
    419419             self.assertEqual(NS_PUBSUB_OWNER, element.configure.uri)
Note: See TracChangeset for help on using the changeset viewer.