Help needed with development of new scraper for filmdelta.se (Swedish Movie Scraper)?

Uhm, whats the current problem with the scraper? The modified version i've attached to the ticket (see above) is working fine for me.

Daniel Malmgren Wrote:edit: Ok, I found the error. They obviously has redone the search result on filmdelta.se since I did my first expression, and I've stupidly enough only tested with a page I saved down for testing. That's why my tests worked and not xbmc :-)

The nature of web-scraping
the page is always bound to change

Ok. Quite small effort involved in fixing that one. Now "Sagan om ringen" actually returns quite a list of hits. My xbmc chose the wrong one though. From the wiki (at http://wiki.xbmc.org/?title=Scrapers I understand that if GetSearchResults returns more than one result the user gets to choose which one is correct. When does this choosing happen? I've never actually seen it

New xml at http://pastebin.com/m648a3981

/Daniel

let me be the first to say the obligatory;

doh!

it happens if you do a manual lookup using 'movie information' from context

vdrfan Wrote:Uhm, whats the current problem with the scraper? The modified version i've attached to the ticket (see above) is working fine for me.

Well, currently I guess everything that I've implemented is working. I haven't included all possible info from filmdelta though. And the thumbs are way to low resolution, I guess I could fetch them from another place (imdb?) instead, right?

Nicezia Wrote:The nature of web-scraping
the page is always bound to change

Yep. I've established a communication with the folks behind filmdelta though, my hope is that they could put up something for me that is not as bound to change. Not sure if it's going to happen though...

/Daniel

spiff Wrote:it happens if you do a manual lookup using 'movie information' from context

Ah. Thanks there mate. I've always wondered

/Daniel

vdrfan Wrote:Uhm, whats the current problem with the scraper? The modified version i've attached to the ticket (see above) is working fine for me.

Darn. Just realized that you actually changed stuff in the xml in the ticket. Thought you just posted my xml right up. Means that I've been working in parallell with you.

I'll incorporate my changes into your xml and post again...

/Daniel

doh! thanks for not ignoring me

not to be thread jacking, but since there's some attention here, what's the accepted indentation on the XML (how many spaces generally do you use per indentation - so i can get the scraper editor outputting to that standard)