Closed (fixed)
Project:
Smileys
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Mar 2006 at 00:38 UTC
Updated:
26 Jul 2006 at 10:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
smilodon commentedHEY, THIS GUYS NEEDS HELP , SO HELP HIM !!!
... please.
Comment #2
fallacious commentedHere's what I get: The 'edit' and 'add' forms display "post"
In a test post
:-) displays as p>Smiling
and
:-( displays as Sad
The filter seems to display the description insteed of the image.
The module file also looked more like a large patch file (there were -/+ lines in it)
Comment #3
smilodon commentedIm not a php coder, but cant u replace the Smile with a html
code ?
All im sayng is thath emoticons is a very important thing ! Ppl are used to express themselves with smileys.
Im sad :( , but my face is 90 degrees lifted, and i hate that !
Comment #4
Lappie commentedYeah, the file attachment is screwed up; part code, part diff.
The issue with the cvs of smileys is that it seems that Drupal has changed the way how relative paths are resolved. I don't know if this is correct nor do I have searched for a reference for it on the site. However, what this means is that you have to make the paths to the smiley images absolute (to a certain extent).
For instance, if you change the example.mysql file to the following and then load in your DB (backup first!), then it should work (YMMV). Notice the initial slash before 'misc/smileys...'??
That's how it worked for me. Also for existing setups this is the easiest way to add the initial slash. Usually this shouldn't break your DB (unless you've added non-standard smileys).
A similar issue was also present for the quote-module. However, for that module you can find a patch lieing around somewhere that resolves all kinds of update issues.
Comment #5
fallacious commentedWith the above database change, the smileys do appear on the page. The article's body beings with p> and the admin pages are still broken. Broken admin pages would be acceptable to me for the purposes of an upgrade to drupal 4.7, provided that the input filter works.
Comment #6
fallacious commentedHere's a version of the module that *seems* to work. YMMV.
If you find any bugs, please let me know. It would be nice if somebody could repackage the module, using the new installer...
Comment #7
fallacious commentedThe previous version made the input filter configs go into an infinite loop. Kludged version appended.
Comment #8
Steven commentedPlease post a proper patch.
Comment #9
fallacious commentedRelative to what? 4.6 or cvs?
Comment #10
fallacious commentedHere's a patch relative to the cvs version.
The code is of "seems to work for me" quality.
Comment #11
desm0n commentedDid anyone ever get this to work ? Does the patch make it fully 4.7 compatable ? I'm not too hot on patches so can anyone provide a full module with install for test ? I'd love to have this filter on my 4.7 drupal.
Comment #12
Anonymous (not verified) commentedhi,
i'd like to inegrate this module into my chatroom module, so i'd like to help with getting this module ready for 4.7.
cheers
justin
Comment #13
Steven commentedAlready has been updated.
Comment #14
(not verified) commented