I began porting version 1.1 of this module to Drupal 7, mostly as an excercise to see what would be involved.
Attached to this issue is a first pass patch, which is NOT pretty or ready yet!
I started by running dictionary through coder_upgrade which does some automatic conversion from Drupal 6 to 7. It generates a report, which is included in the patch as a new file "REVIEW-Drupal-7.txt". It also adds a ton of comments to the source about things that need to be checked or fixed. I have left all the notes that I haven't checked or fixed yet (this is the main source of ugliness in the patch).
I also updated the README.txt to reflect the new way admin is presented to users and started collecting some notes about the conversion in a new file called "NOTES-Drupal-7.txt". Right now it only contains the following note:
* The 'cck' module is still needed (even though most of it is now in core) because it adds
the ability to use "Allowed values PHP code" for select fields. (At the time of this writing
there has been no official release, so you have to get 7.x-2.x-dev)
I've managed to get the following things working:
- Module installs and sets up the "dictionary_term" content type and the "dictionary_term_iid" field.
- You can add/edit/remove languages
- You can add/edit/remove terms
- You can add/edit/remove translations
Here are things that definitely aren't working:
- The page at /dictionary
- Even though the field creates fine, and looking at the database it seems like allowed_values_php got in there, it doesn't seem to actually work
And there is lots of other stuff that hasn't even been tested!
Any future work I do on this will be posted as new patches in the comments on this issue.
However, for the moment, I feel like I've learned all that I wanted to about porting this module. So, I probably won't return to it for awhile. But if someone else wants to continue working on the Drupal 7 port, this patch should give them a head start!
Best Regards,
David.
| Comment | File | Size | Author |
|---|---|---|---|
| dictionary-drupal7-01.patch | 48.26 KB | dsnopek |
Comments
Comment #1
dropbydrop commentedsub
Comment #2
zmeharen commentedsub
Comment #3
davidam commentedI'm interested in this subject. Are there plans to work on it?