Closed (fixed)
Project:
Services
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
16 Jun 2011 at 14:48 UTC
Updated:
28 Jul 2011 at 01:43 UTC
Jump to comment: Most recent file
Comments
Comment #1
robin van emden commentedPatch, partly based on http://drupal.org/node/1086526#comment-4201390
Comment #2
robin van emden commentedComment #3
robin van emden commentedSmall formatting fix.
Comment #4
kylebrowning commentedWhy are you not using services_resource_uri?
EDIT: woops should have read the first post.
Comment #5
kylebrowning commentedThis is a bug in amfserver i believe, it works for me, AND the tests pass for me. I tested in php 5.2 and 5.3
https://skitch.com/kylebrowning/ffn7f/untitled
Comment #6
robin van emden commentedThanks. I will continue my investigation into this issue.
Comment #7
rolf vreijdenberger commentedHi Robin,
nice to see you involved again on the drupal side of things. Do you have an update on this issue?
Comment #8
robin van emden commentedHi Rolf,
I was away for a few days, finally found the time to dig a little deeper.
I am still unconvinced this is a definite bug in Amfserver. If so, the same bug can be found in the XMLRPC server.
The problem is the need for the services_resource_uri function to return an uri , if the taxonomy_service_select_nodes function is to return anything.
The services_resource_uri function is found in services.runtime.inc, where it looks for a uri formatter through services_get_server_info, which otherwise returns null:
This works fine IF the server (REST, XMLRPC, AMF) has defined this server value first, for example in RESTServer.inc:
The patch circumvented the services_resource_uri function, which made sure the function returned a value, instead of null.
So either:
There has been some discussion on this subject here: #1086526: Missing url path / url alias
Since the choices mentioned have broader implications than amfserver, I think it is prudent I reassign the this issue to the Services module again.
Comment #9
robin van emden commentedComment #10
kylebrowning commentedOk, is this also broken in drupal 6?
Comment #11
robin van emden commentedFound some time to test this for XMLRPC server in D6, and the answer is no, this is not broken in the same way in D6.
In D6, taxonomy_term.selectNodes does not make use of services_resource_uri - it can be found in other resources (file, comment, node, user), but I have not had time to test these service in XMLRPC.
I did encounter a problem for taxonomy_term.selectNodes when using XMLRPC, a quick fix was changing
for
but that is a different issue.
Comment #12
kylebrowning commentedI see the issue, servers not required to have a resource uri formatter, however you probably should so that you can take advantage of calling that URI to load a specific entity.
Im running a test on this fix, and if it works ill fix it across the board and close this ticket.
Comment #13
kylebrowning commentedOk it worked, this has been fixed in dev.
My solution was much simpler, move the node_load outside of the resource uri_formatter.
Comment #14
kylebrowning commented