can you please make the module for drupal 5?
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | delicious-5x.patch | 13.59 KB | douggreen |
| #9 | delicious_5.patch | 25.52 KB | ax |
| #4 | delicious_4.patch | 25.09 KB | ax |
| #3 | delicious_2_0.patch | 10.38 KB | ax |
| #2 | delicious_1.patch | 6.65 KB | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) commentedwould you please test this patch .
Comment #2
Anonymous (not verified) commentedhere is the complete patch , please provide feedback
Comment #3
ax commentedmade this actually work for 5.X:
* provided info-file - without this, module cannot be enabled
* fixed form submission at user/<uid>/delicious
* made block links work
* made "smart tagging" work, with a big regexp instead of so many foreaches
* doc, coding-style, redundancy
this module needs more work. see TODO: in code, and just look at it.
Comment #4
ax commentedmore fixes and enhancements, incl.
* support for private links (only show them to owning user)
* better recent bookmarks formatting
* better and tested api doc
* and more, which i forgot because its 3 month ago i did this
may someone find this useful.
Comment #5
derekvan commentedHi,
I can't get this patch to work--many hunks fail. Any chance of getting it re-synced? Or maybe I'm doing it wrong? I executed this command:
patch delicious.module < delicious_4.patch
Thanks!
Comment #6
ax commentedyou need to use
patch -p0 < delicious_4.patch, see Applying patches in the Developing for Drupal handbook.Comment #7
derekvan commentedthanks for the reply--I'm still having trouble after reading the guide. Here's what I'm doing:
navigating to: modules/my_modules/delcious
patch -p0 < delicious_4.patch
I didn't get any hunk failures this time. However, the delicious module doesn't show up in my list of modules, so I can't enable it. I tried deleting the delicious directory and starting over, by first applying the delicious_0.patch, and the the delicious_1.patch, but this also resulted in lots of errors.
Can you walk me through the process here? Thanks so much,
derek
Comment #8
ax commentedyou only need delicious_4.patch. make sure there is only one delicious.module in your drupal install and only one entry for it in the system table and that its path is right. you are running drupal 5, right? check google. if you really cannot get it to work, pls contact me off site (axel AT kollmorgen DOT net , skype:akollm, icq:19144959).
Comment #9
ax commentedsorry, my wrong. delicious_4.patch misses the info-file from delicious_2_0.patch. here is a new one including everything needed - hope it will work now.
Comment #10
ax commentedfurther fixes, updates, and development in my sandbox
Comment #11
Anonymous (not verified) commentedI have a corrected a few problems .
1. Updating delicious bookmarks was not working .
2. The number of delicious links for a given tag was not correct
3. A few other minor things
Please download the full delicous module from here
http://paulbooker.co.uk/?q=delicious-5.x-1.0-dev-RC2
for testing If i have time later i will generate a patch.
Comment #12
ax commentedpaul - there is nothing to download at the link you provided. please submit patches here or to the Drupal contrib CVS repository. thanks!
Comment #13
Anonymous (not verified) commentedThanks Ax ,
I will upload the patch later today , you can now login with your drupal account on my site and download the module in the meantime if you wish .
Speak to you soon .
Paul
Comment #14
douggreen commentedI also have significant changes for a 5.x patch. Attached is my patch.
What we need to do is get CVS rights to this project and get them committed.
Comment #15
douggreen commentedI just realized that I have enough rights on d.o to give myself CVS rights to the project, so I did, I created a new 5.x branch, applied my patch, and created a release. Earl is still looking for a project maintainer, and I'm not offering to do that. I simply pushed this project forward a little...
Comment #16
merlinofchaos commentedI found a project maintainer. It's just that the project maintainer is awol. Go figure.
I guess we need Yet Another Project Maintainer for this.
Comment #17
ax commentedmerlinofchaos: count me in as YAPM for this.
Comment #18
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.