Do not import base namespaces about soapfish HOT 5 OPEN

@oskrkal: might this be addressed by your xsd-resolver branch?

from soapfish.

@pope1ni: I'm sorry for late response, I somehow missed your comment.

I'm not sure if I understand the problem correctly. Is it that the schemaLocation attribute is not mandatory and only the namespace is sometimes specified in the import attribute? If so, the XSDCachedSchemaResolver indeed could address this issue. It allows to pre-define schema location for specific namespaces and is then able to resolve the schema purely by the namespace, if location is not specified.

Another feature of the resolver is that it caches all schemas, so it can save some time when, for example, a WSDL file imports a lot of schemas, most of which import a common set of other schemas.

The branch should be ready for merging into upstream. I just thought I would go through it and fix some of the constructions that you fixed in my last pull request, but I have been quite busy at work recently.

from soapfish.

@iurisilvio: can you clarify - my understanding from what you have written was that these shouldn't be imported at all?

from soapfish.

Yes, I think soap/envelope and soap/encoding should be ignored. We implement them internally.

I have some third-party WSDLs with these imports, I can't use soapfish with them. Even if the generated code is not broken, it is still a lot of useless code.

These namespaces are in https://github.com/soapteam/soapfish/blob/master/soapfish/namespaces.py#L22, maybe we can use it to define these "ignored schemas".

from soapfish.

All right, thanks for clarification. Then I think XSDSchemaResolver/XSDCachedSchemaResolver in my xsd-resolver branch is not the right place to address this problem. Its purpose is simply to load a schema file from correct location, parse it and return it to the caller, or to return a cached xsdspec.Schema instance if the schema has already been processed.

We probably need to somehow preload internally implemented schemas into known_paths list and ensure that get_type function returns correct module and class name.

from soapfish.