Posted by chizuko on November 3, 2010 at 12:31am
4 followers
| Project: | iTunes |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Issue Summary
We needed rss that complies with iTunes U specification at Presidetn’s Office site at Arizona State University:
http://deimos.apple.com/rsrc/doc/UsingiTunesUPublicSiteManager/AboutTheF...
Everything worked well with iTunes module as it is, except we needed within tag.
So, I modified itunes.module and itunes.install. The modified module is working well at our website.
I’m new to do module work and I’m very excited to submit this patch and give back to Drupal community. I want to thank Jeff Beeman for helping me with this project.
It would be great if you could incorporate this patch in the next version of iTunes module.
If you have any questions, please let me know.
Thank you !
Comments
#1
Attached is the patch for iTunes U.
In the above post, under Description, in the 3rd line, I wanted to write:
..., except we needed
<itunesu:category itunesu:code=" ">within<item>tag.#2
Thanks for the patch. The one problem that jumps out at me is the name itunesu_categories(), a module needs to stick within its namespace so it would need to be called something like itunes_u_categories() or itunes_university_categories() so we don't need to worry about conflicting with an itunesu module (if someone creates such a thing).
Also I'm not sure the switch for $GLOBALS['db_type'] is necessary. The schema API's db_add_field() should abstract out the details.
Also for future reference here's the Apple docs: http://deimos.apple.com/rsrc/doc/iTunesUAdministrationGuide/AddingConten...
#3
Hi drewish,
Thank you for your feedback. I will look at it and work on it during Christmas/New Years holidays.
p.s. I'm sorry for delayed response.
#4
Subscribing. I am also interested in how to add itunes u tags with this module.