Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 CreditAttribution: 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 CreditAttribution: ilo commentedmm I forgot to remove some globals. Also I have include more error checking.
Comment #3
ilo CreditAttribution: ilo commentedsome minor changes to the patch.
Comment #4
ilo CreditAttribution: 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 CreditAttribution: ilo commentedCommited to 6.x-1.x-dev branch.
Comment #7
lpalgarvio CreditAttribution: lpalgarvio commentedsupport for Libraries directory or Libraries API would be a good thing