Merge lp:~zorba-coders/zorba/bug-1169191 into lp:zorba

Proposed by Chris Hillery
Status: Merged
Approved by: Markos Zaharioudakis
Approved revision: 11373
Merged at revision: 11372
Proposed branch: lp:~zorba-coders/zorba/bug-1169191
Merge into: lp:zorba
Diff against target: 32 lines (+6/-5)
2 files modified
src/context/static_context.cpp (+6/-4)
test/fots/CMakeLists.txt (+0/-1)
To merge this branch: bzr merge lp:~zorba-coders/zorba/bug-1169191
Reviewer Review Type Date Requested Status
Markos Zaharioudakis Approve
Chris Hillery Approve
Review via email: mp+159035@code.launchpad.net

Commit message

Only attempt to resolve a base URI against the entity-retrieval URI if said entity-retrieval URI itself absolute.

To post a comment you must log in.
Revision history for this message
Chris Hillery (ceejatec) :
review: Approve
lp:~zorba-coders/zorba/bug-1169191 updated
11373. By Chris Hillery

One less FOTS failure.

Revision history for this message
Markos Zaharioudakis (markos-za) :
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 bug-1169191-2013-04-16T06-49-41.896Z 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
=== modified file 'src/context/static_context.cpp'
--- src/context/static_context.cpp 2013-04-02 21:43:27 +0000
+++ src/context/static_context.cpp 2013-04-15 22:44:26 +0000
@@ -1499,10 +1499,12 @@
1499 if (found)1499 if (found)
1500 {1500 {
1501 URI base(entityUri);1501 URI base(entityUri);
1502 URI resolvedURI(base, userBaseUri);1502 if (base.is_absolute()) {
1503 theBaseUriInfo->theBaseUri = resolvedURI.toString();1503 URI resolvedURI(base, userBaseUri);
1504 theBaseUriInfo->theHaveBaseUri = true;1504 theBaseUriInfo->theBaseUri = resolvedURI.toString();
1505 return;1505 theBaseUriInfo->theHaveBaseUri = true;
1506 return;
1507 }
1506 }1508 }
15071509
1508 URI base(get_implementation_baseuri());1510 URI base(get_implementation_baseuri());
15091511
=== modified file 'test/fots/CMakeLists.txt'
--- test/fots/CMakeLists.txt 2013-04-15 13:37:43 +0000
+++ test/fots/CMakeLists.txt 2013-04-15 22:44:26 +0000
@@ -488,7 +488,6 @@
488EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof12 0)488EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof12 0)
489EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof14 0)489EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof14 0)
490EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof15 0)490EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof15 0)
491EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof16 0)
492EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof17 0)491EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof17 0)
493EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof18 0)492EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof18 0)
494EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof19 0)493EXPECTED_FOTS_FAILURE (misc-HigherOrderFunctions xqhof19 0)

Subscribers

People subscribed via source and target branches