DRUPAL-7 version of vcard. done!
dman - October 23, 2009 - 14:05
| Project: | vCard |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | dman |
| Status: | closed |
Jump to:
Description
Here we go. I thought this one would be a good place to start. Turned out to be a little trickier that I thought, but only a little.
Attached is a patch that will apply to 6-dev, or a full replacement to put into a new branch.
I started poking at this initially to see if I could expose some of the new RDFa things in D7. I couldn't, but it was worth the exercise.
Changelog
dman 2009-10 : Upgraded to DRUPAL-7
Cleaned up to meet coder.module requirement, with the help of the coder
upgrade review.
Moved the admin menu item to admin/config/people/vcard "People and
permissions" next to profile and user accounts.
Updated to D7 profile.module database schema changes.
Adjusted Admin form help text.
Updated hook_user_view to d7 arguments
Changed the indexing of vcard_get map to key by profile field names, not
profile field IDS - to match the way profile.module exposes the data.
Incorporates cleanup changes from [63reasons-AT]
http://drupal.org/node/552204| Attachment | Size |
|---|---|
| vcard-upgrade_to_DRUPAL7-20091024.patch | 16.09 KB |
| vcard-DRUPAL7.tar_.gz | 6.53 KB |
#1
Hi Dan,
that was a great kickstart, thanks.
I commited your version to HEAD for further development.
#2
Please check out the latest version.
* Some additional clean-up
* Changed the way the vCard is generated
I wanted a central point, where vcard data is assembled.
That data is later used by the template for hCard display and by the vCard builder to generate the vCard for download
* hook_theme update
Changed a few days ago
* Removed the theme_vcard()
From my point of view it's redundant, as we have a template file now.
* Added template_preprocess_vcard()
To have the variables prepared outside the template file
* Cleaned up the template file
Please check it out. Feedback welcome!
#3
After some testing, I think this should be ok.
Added a DRUPAL-7 BRANCH and a DEV version.
Thanks for your help!
#4
Automatically closed -- issue fixed for 2 weeks with no activity.