Closed (fixed)
Project:
SOAP Client
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
18 Sep 2009 at 11:18 UTC
Updated:
12 Sep 2010 at 09:59 UTC
Jump to comment: Most recent file
This part is missing in the interface. If nuSOAP is not to be installed with the module by default, there should be a place to configure it.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | soapclient.make-nusoap-library-configurable_0_0.patch | 11.93 KB | ilo |
| #3 | soapclient.make-nusoap-library-configurable.patch | 12.01 KB | ilo |
| #2 | soapclient.make-nusoap-library-configurable.patch | 12.02 KB | ilo |
| #1 | soapclient.make-nusoap-library-configurable.patch | 10.9 KB | ilo |
Comments
Comment #1
ilo commentedI though it would be easier, but it took 11kb to do the change. I have included some new error checking because of the change (it could happend that library is configured but deleted, or is installed but not configured, or is not installed neither configured, or installed but configured in a wrong directory). To make requirements checks available during installation I included the .install file, so I also created the uninstall hook to remove the module variables.
The module looks for installed nusoap files within sites/all and sites/%current_site, in the hope of finding at least one version. In the documentation it will be a good point to use /sites/all/plugins to install nuSOAP library there, so other modules may still use it, not having to install nuSOAP twice.
I just left the hook_init to make cyphering available for now, but I guess this will be removed. Cyphering, even if used by some services has nothing to do with soapclient module.
Once this patch gets reviewed and commited, the next steps will be:
#408360: NuSOAP project code should not be in CVS
#581060: Cyphering is not being in use
#408362: Directory structure not correct
Comment #2
ilo commentedmm I forgot to remove some globals. Also I have include more error checking.
Comment #3
ilo commentedsome minor changes to the patch.
Comment #4
ilo commentedPatch reviewed by Peter Bex:
- English typo: any to none
- Preference is given to sites/%current_site and then to sites/all when searching for installed nuSOAP Libraries
Comment #5
ilo commentedCommited to 6.x-1.x-dev branch.
Comment #7
lpalgarvio commentedsupport for Libraries directory or Libraries API would be a good thing