Merge lp:~zorba-coders/zorba/bug-1189807-html into lp:zorba/html-module

Proposed by Chris Hillery on 2013-07-24
Status: Merged
Merged at revision: 93
Proposed branch: lp:~zorba-coders/zorba/bug-1189807-html
Merge into: lp:zorba/html-module
Diff against target: 25 lines (+6/-2)
1 file modified
test/Queries/link_crawler2.xq2 (+6/-2)
To merge this branch: bzr merge lp:~zorba-coders/zorba/bug-1189807-html
Reviewer Review Type Date Requested Status
Luis Rodriguez Gonzalez Approve on 2013-08-07
Chris Hillery Approve on 2013-07-24
Review via email: mp+176636@code.launchpad.net

Commit message

Fixes for new XML module.

To post a comment you must log in.
Chris Hillery (ceejatec) :
review: Approve
review: Approve
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue succeeded - proposal merged!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'test/Queries/link_crawler2.xq2'
2--- test/Queries/link_crawler2.xq2 2012-08-04 02:23:35 +0000
3+++ test/Queries/link_crawler2.xq2 2013-07-24 09:42:57 +0000
4@@ -17,7 +17,8 @@
5 import module namespace http = "http://www.zorba-xquery.com/modules/http-client";
6 import module namespace map = "http://www.zorba-xquery.com/modules/store/data-structures/unordered-map";
7 import module namespace html = "http://www.zorba-xquery.com/modules/converters/html";
8-import module namespace parse-xml = "http://www.zorba-xquery.com/modules/xml";
9+import module namespace x = "http://zorba.io/modules/xml";
10+import schema namespace opt = "http://zorba.io/modules/xml-options";
11 import module namespace file = "http://expath.org/ns/file";
12
13 declare namespace an = "http://www.zorba-xquery.com/annotations";
14@@ -218,7 +219,10 @@
15 map:insert($local:processed-internal-links, (<MESSAGE>{concat("cannot tidy: ", $err:description)}</MESSAGE>,
16 <RESULT>broken</RESULT>), $x);
17 try{
18- $content:=parse-xml:parse-xml-fragment ($string-content, "");
19+ $content:=x:parse($string-content,
20+ <opt:options>
21+ <opt:parse-external-parsed-entity/>
22+ </opt:options>);
23 }
24 catch *
25 { map:insert($local:processed-internal-links, <MESSAGE>{concat("cannot parse: ", $err:description)}</MESSAGE>, $x);}

Subscribers

People subscribed via source and target branches

to all changes: