symphonycms / remote_datasource

An improved datasource to fetch data from TXT, CSV, XML and JSON sources.
Other
14 stars 9 forks source link

Transformed XML is invalid with Instagram API #8

Closed nitriques closed 10 years ago

nitriques commented 11 years ago

I have the same error line twice:

loadXML(): Start tag expected, '<' not found in Entity, line: 1

Will investiguate more on that...

nitriques commented 11 years ago

BTW this is the failling call: https://api.instagram.com/v1/tags/ggi/media/recent?client_id=XXXXX

nitriques commented 11 years ago

Ok the problems seems to be related to the fact that if you change the XPath filter, you must clear the cache. After clearing the cache, the bug is gone.

I still think this should be addressed though.

brendo commented 11 years ago

Interesting, the cache key includes the XPath value, so changing the XPath should result in a new cache being created.

One to investigate.

nitriques commented 11 years ago

Yeah that's right. The thing is, the link you gave me is a link to the cloudstoragefield extension.. Is that normal ?

BTW, I will need this field soon. Is S3 supported yet ? If not, I will contribute that.

brendo commented 11 years ago

Odd, sorry, I've updated the link, https://github.com/symphonycms/remote_datasource/blob/master/data-sources/datasource.remote.php#L142-L177

BTW, I will need this field soon. Is S3 supported yet ? If not, I will contribute that.

No, not yet, the client project only requires Rackspace. I had thought of mimicking something similar to the oEmbed field to enable other providers though so your help would be appreciated!

nitriques commented 10 years ago

I do not have this issue anymore so I think it's ok to close it.