Hi,
We would like to help you with porting Feeds SOAP fetcher to Drupal 7.
Have you started working on it (If yes, can you upload a dev version?)
If not, we'd love to start it.

Comments

alond’s picture

subscribing

ruifigueiredo’s picture

subscribing

kasperg’s picture

We're also interested in a Drupal 7 port.

I've found this sandbox but it forks the original code by Cyberwolf - not this project.

Does anyone else have ports in sandboxes or elsewhere that they would like to share?

stefan.norman’s picture

StatusFileSize
new9.78 KB

I have ported this project to Drupal 7, but I don't know how to proceed to get it up as 7.x-1.x-dev. I'm attaching the code. It's based on the 6.x-1.x-dev.tar.gz.

kasperg’s picture

#4: Please attach it as a patch against the current codebase.

Only the maintainer can create new branches/versions for the project.

stefan.norman’s picture

StatusFileSize
new4.83 KB

Ok, here goes. Patching 6.x-1.x.
Thanks kasperg!

Daniel Ferreira Jorge’s picture

Hi, does anybody know how to use this to connect with a WS that need a digital certificate? (PKCS12)
Thanks!

stefan.norman’s picture

Status: Active » Needs review
kasperg’s picture

StatusFileSize
new9.63 KB

I've updated the patch in #6 with the following changes:

  • Remove autoload dependency (as in #6), added required core version and keep files[] in info file. #6 did not apply for me here.
  • Implement ctools plugin hooks (As in #6). I'm unsure if this is needed as the module already implements hook_feeds_plugins().
  • Use FeedsFetcher directly instead of FeedsImportBatch (as in #6).
  • Update WsSoapClient->__doRequest() signature to fix method override. WsSoapClient->__doRequest() was not called without this change.
  • Use call_user_func() instead of __soapCall(). WsSoapClient->__doRequest() was not called without this change.
  • Comply with Drupal code standard.

#7: You can pass the path to a certificate as an option when creating the SoapClient object. Currently this module does not support certificates. Please create a separate issue for this.

lgomezma’s picture

Any chance of #9 getting an official Drupal 7 version? Already implemented it in a Drupal 7 installation without problems.

jerry’s picture

Status: Needs review » Reviewed & tested by the community

It's working well for me, too.

It'd be nice to have a more generalized configuration option to set custom headers; I've had to hack in some custom auth stuff for my own application. I'm probably not familiar enough with SOAP to know the cleanest way to approach this, unfortunately.

liquidcms’s picture

Category: Task » Feature request
Issue summary: View changes
Status: Reviewed & tested by the community » Needs work

i used patch in #9 to patch 6.x-1.x and when i create change Fetcher to SOAP on new Feed importer i get:

Missing Feeds plugin FeedsSOAPFetcher.

Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'FeedsMissingPlugin_feeds_form' not found or invalid function name in drupal_retrieve_form() (line 841 of E:\www\mma\includes\form.inc).

liquidcms’s picture

this code was wrong:

/**
 * @file
 * Feeds SOAP - SOAP fetcher for Feeds.
 */

/**
 * Implements hook_ctools_plugin_api().
 */
function feeds_soap_ctools_plugin_api($owner, $api) {
  if ($owner == 'feeds' && $api == 'plugins') {
    return array('version' => 1);
  }
}

should be $owner == 'feeds_soap'

fnapo’s picture

Ehy! Did you solve the "Missing Feeds plugin FeedsSOAPFetcher." ? Changing the $owner still give me the error.
It's a pity I can't use this module.