Closed (fixed)
Project:
vCard
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
23 Oct 2009 at 14:05 UTC
Updated:
12 Nov 2009 at 10:40 UTC
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
| Comment | File | Size | Author |
|---|---|---|---|
| vcard-DRUPAL7.tar_.gz | 6.53 KB | dman | |
| vcard-upgrade_to_DRUPAL7-20091024.patch | 16.09 KB | dman |
Comments
Comment #1
sanduhrsHi Dan,
that was a great kickstart, thanks.
I commited your version to HEAD for further development.
Comment #2
sanduhrsPlease 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!
Comment #3
sanduhrsAfter some testing, I think this should be ok.
Added a DRUPAL-7 BRANCH and a DEV version.
Thanks for your help!