Merge lp:~zorba-coders/zorba/fix_xqdoc_oauth_module into lp:zorba/oauth-module

Proposed by Sorin Marian Nasoi
Status: Merged
Approved by: Cezar Andrei
Approved revision: 22
Merged at revision: 22
Proposed branch: lp:~zorba-coders/zorba/fix_xqdoc_oauth_module
Merge into: lp:zorba/oauth-module
Diff against target: 101 lines (+24/-13)
2 files modified
src/com/zorba-xquery/www/modules/oauth/client.xq (+18/-1)
src/com/zorba-xquery/www/modules/oauth/error.xq (+6/-12)
To merge this branch: bzr merge lp:~zorba-coders/zorba/fix_xqdoc_oauth_module
Reviewer Review Type Date Requested Status
Cezar Andrei Approve
Sorin Marian Nasoi Approve
Review via email: mp+103377@code.launchpad.net

Commit message

- added XQDoc comments for some parameters in client.xq
- removed unused function in error.xq and corrected some typo's

Description of the change

- added XQDoc comments for some parameters in client.xq
- removed unused function in error.xq and corrected some typo's

To post a comment you must log in.
Revision history for this message
Sorin Marian Nasoi (sorin.marian.nasoi) :
review: Approve
Revision history for this message
Cezar Andrei (cezar-andrei) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job fix_xqdoc_oauth_module-2012-04-24T22-09-15.025Z is finished. The final status was:

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/com/zorba-xquery/www/modules/oauth/client.xq'
2--- src/com/zorba-xquery/www/modules/oauth/client.xq 2012-04-12 08:45:03 +0000
3+++ src/com/zorba-xquery/www/modules/oauth/client.xq 2012-04-24 20:55:43 +0000
4@@ -92,6 +92,16 @@
5 : href="https://api.twitter.com/oauth/access_token"/>
6 : </sp:service-provider>
7 : </pre>
8+ : @param $consumer-key Client Identifier, also known as the consumer-key
9+ : @param $consumer-secret Client Shared-Secret, also known as the consumer-secret
10+ : @param $signature-method Method with which the signing key is signed (typically HMAC-SHA1)
11+ : @param $realm Realm that defines the protection space
12+ : @param $authorize-url authorize-url
13+ : @param $request-token-method request-token-method
14+ : @param $request-token-url request-token-url
15+ : @param $request-token-callback-url request-token-callback-url
16+ : @param $access-token-method access-token-method
17+ : @param $access-token-url access-token-url
18 : @return instance of the OAuth service provider XML schema.
19 :)
20 declare function oauth:service-provider(
21@@ -137,6 +147,8 @@
22 : <p:parameter name="oauth_token" value="#" />
23 : </p:parameters>
24 : </pre>
25+ : @param $name parameter name
26+ : @param $value parameter value
27 : @return instance of the OAuth parameters XML schema.
28 :)
29 declare function oauth:parameters($name as xs:string, $value as xs:string)
30@@ -167,6 +179,9 @@
31 : <p:parameter name="oauth_token_secret" value="#" />
32 : </p:parameters>
33 : </pre>
34+ : @param $parameters parameters
35+ : @param $name parameter name
36+ : @param $value parameter value
37 : @return instance of the OAuth parameters XML schema.
38 :)
39 declare function oauth:add-parameter($parameters as schema-element(p:parameters)?, $name as xs:string, $value as xs:string)
40@@ -276,6 +291,7 @@
41 : This function is provided for convenience.
42 : @see http://tools.ietf.org/html/rfc5849#section-2.3
43 : @param $service-provider Contains service provider information
44+ : @param $parameters parameters
45 : @return token credentials correctly parsed as parameter elements, or an error if http response status is not 200 OK
46 :
47 : <pre class="ace-static" ace-mode="xquery">
48@@ -318,7 +334,8 @@
49 : This function is provided for convenience.
50 : @see http://tools.ietf.org/html/rfc5849#section-3
51 : @param $protected-resource (Not schema-validated) http:request element with http method and href.
52- : @param $request request element containing the client's request
53+ : @param $service-provider Information about the service provider
54+ : @param $parameters parameters
55 : @return protected resources parsed as parameter elements, or an error if http response status is not 200 OK
56 :
57 : <pre class="ace-static" ace-mode="xquery">
58
59=== modified file 'src/com/zorba-xquery/www/modules/oauth/error.xq'
60--- src/com/zorba-xquery/www/modules/oauth/error.xq 2012-04-11 20:22:31 +0000
61+++ src/com/zorba-xquery/www/modules/oauth/error.xq 2012-04-24 20:55:43 +0000
62@@ -34,32 +34,26 @@
63 declare variable $oerr:errNS as xs:string := "http://www.zorba-xquery.com/modules/oauth/errors";
64
65 (:~
66- : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'err:OC001'. This signing method is not implemented yet.
67+ : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'OC001'. This signing method is not implemented yet.
68 :)
69 declare variable $oerr:OC001 as xs:QName := xs:QName("oerr:OC001");
70
71 (:~
72- : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'err:OC002'. This signing method is not supported.
73+ : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'OC002'. This signing method is not supported.
74 :)
75 declare variable $oerr:OC002 as xs:QName := xs:QName("oerr:OC002");
76
77 (:~
78- : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'err:OC003'. Http 401 error.
79+ : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'OC003'. Http 401 error.
80 :)
81 declare variable $oerr:OC003 as xs:QName := xs:QName("oerr:OC003");
82
83 (:~
84- : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'err:OC004'. Http 500 error.
85+ : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'OC004'. Http 500 error.
86 :)
87 declare variable $oerr:OC004 as xs:QName := xs:QName("oerr:OC004");
88
89 (:~
90- : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'err:OC004'. Http 500 error.
91+ : xs:QName with namespace URI="http://www.zorba-xquery.com/modules/oauth/errors" and local name 'OC004'. Http 500 error.
92 :)
93-declare variable $oerr:OC005 as xs:QName := xs:QName("oerr:OC005");
94-
95-declare function oerr:handle($a, $b, $c, $d)
96-{
97- ($a, $b, $c, $d)
98-};
99-
100+declare variable $oerr:OC005 as xs:QName := xs:QName("oerr:OC005");
101\ No newline at end of file

Subscribers

People subscribed via source and target branches

to all changes: