Skip to content

Fixed Chameleon schema support and added support for an edgecase with unqualified elements.#888

Merged
mvantellingen merged 1 commit intomvantellingen:masterfrom
apollo13:issue887
Dec 17, 2018
Merged

Fixed Chameleon schema support and added support for an edgecase with unqualified elements.#888
mvantellingen merged 1 commit intomvantellingen:masterfrom
apollo13:issue887

Conversation

@apollo13
Copy link
Copy Markdown
Contributor

This partially reverts the recently introduced support for chameleon
schemas in the sense that it doesn't consider the targetNamespace of the
WSDL as default namespace for the schema. Sadly it is valid (while ugly
and not widely used) to have schemas with no namespace.

Secondly this commits allows refering to those unqualified elements if
an explicit empty (sic) is found. This seems to be mostly in
line with https://www.w3.org/TR/2012/REC-xmlschema11-1-20120405/#src-resolve
(if I read it correctly).

… unqualified elements.

This partially reverts the recently introduced support for chameleon
schemas in the sense that it doesn't consider the targetNamespace of the
WSDL as default namespace for the schema. Sadly it is valid (while ugly
and not widely used) to have schemas with no namespace.

Secondly this commits allows refering to those unqualified elements if
an explicit empty <import/> (sic) is found. This seems to be mostly in
line with https://www.w3.org/TR/2012/REC-xmlschema11-1-20120405/#src-resolve
(if I read it correctly).
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 14, 2018

Codecov Report

Merging #888 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #888      +/-   ##
==========================================
+ Coverage   89.38%   89.38%   +<.01%     
==========================================
  Files          65       65              
  Lines        4824     4825       +1     
  Branches      841      842       +1     
==========================================
+ Hits         4312     4313       +1     
  Misses        331      331              
  Partials      181      181
Impacted Files Coverage Δ
src/zeep/xsd/schema.py 92.63% <100%> (ø) ⬆️
src/zeep/wsdl/parse.py 91.66% <100%> (ø) ⬆️
src/zeep/wsdl/wsdl.py 94.65% <100%> (-0.03%) ⬇️
src/zeep/xsd/visitor.py 91.26% <100%> (+0.04%) ⬆️

1 similar comment
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 14, 2018

Codecov Report

Merging #888 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #888      +/-   ##
==========================================
+ Coverage   89.38%   89.38%   +<.01%     
==========================================
  Files          65       65              
  Lines        4824     4825       +1     
  Branches      841      842       +1     
==========================================
+ Hits         4312     4313       +1     
  Misses        331      331              
  Partials      181      181
Impacted Files Coverage Δ
src/zeep/xsd/schema.py 92.63% <100%> (ø) ⬆️
src/zeep/wsdl/parse.py 91.66% <100%> (ø) ⬆️
src/zeep/wsdl/wsdl.py 94.65% <100%> (-0.03%) ⬇️
src/zeep/xsd/visitor.py 91.26% <100%> (+0.04%) ⬆️

@mvantellingen
Copy link
Copy Markdown
Owner

Thanks for the follow up. Reading the link you provided and since it seems to work for you i'm going to merge this :-)

Thanks again for the work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants