I used your module as the basis of the Atlassian Crowd SSO module (http://drupal.org/project/crowd). One of the requests that I got was to change the location of the cached WSDL file from /tmp to the files directory. People were concerned that the file might get cleared out of /tmp, which would cause loss of functionality.

Comments

mtolmacs’s picture

mtolmacs commented
Assigned: Unassigned » mtolmacs

You're absolutely right about the possible loss of functionality issue. Albeit I'm concerned to put the WSDL file in the files directory. That is for permanent storage of files managed by Drupal's upload module.

What probably could be the best course of action is to re-pull the wsdl file when it's missing. I will add this to the module.

As a sidenote, caching locally the WSDL was a hack around some PHP issue parsing the JIRA WSDL remotely. It just fails miserably. Curious though if I download it and use it that way it works perfecly.

Anyway, thank you for the report!

ebeyrent’s picture

ebeyrent commented

I agree with you - it's definitely a hack that hopefully can be removed in the future.

In the meantime, if the module requires the cached wsdl in order to work, then in essence, it should be treated as a permanent file, imo.

caiovlp’s picture

caiovlp commented

This WSDL is now being pulled in case it's missing. There is also the alternative to use NuSOAP library in case one is having issues with saving the WSDL file. This was included in 6.x-1.0-beta2.

caiovlp’s picture

caiovlp commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.