|
From: Jason P. <jas...@ho...> - 2001-09-26 01:33:57
|
Hi, We can just add a new s_attribute_type of TRAILER and link it to DVD,VHS,VCD via the s_item_attribute_type table. It is all user data now. No need to modify the item table ever again. (Fingers crossed!) With the move of IMDB to a plugin (OpenDb 0.13), it is just a matter of parsing for the trailer in the imdb plugin and passing the trailer urlparam to the item_input.php for adding to the database. I already have modified item_details.php to request that the imdb plugin control the display of the imdb site item link, via a call to the new site/functions.php get_item_link_block($site_type, $item_id) function. It should a small matter of modifying the imdb plugin config.php $site_item_link_block variable to also show a link to the trailer. I will add the TRAILER type to the 0.13 release, but I will leave it to you to modify the imdb site plugin to actually pass it to item_input.php. Cheers Jason -- Vegan Philosopher Programmer >From: Marc Powell <MP...@ac...> >To: 'Jason Pell' <jas...@ho...> >Subject: RE: [Dvddbp-develop] OpenDb-0.13 coming soon. > >Hi Jason- > >Sorry, I've been pretty busy this week so I haven't much time to carefully >look over everything. I installed a copy of 0.12 on my server and played >around with it a bit. Everything seemed to be running well from what I >could tell. > >You've really done a good job, and quite a massive amount of work went into >it. > >If I can throw in a request while you are working on the db structure, I >was >thinking of introducing a feature to pull a trailer link from the imdb for >movies, similar to what is done with the images. Since we are introducing >a >new database structure, it might be best to try and integrate it now when >everything is new-- even if the db field is reserved for future use. This >is of course, assuming you approve of implementing this option. > >Keep up the good work :) > >-- Marc > _________________________________________________________________ Get your FREE download of MSN Explorer at http://explorer.msn.com/intl.asp |