I've very recently upgraded from Drupal 4.6 to 5.1 (laregely motivated by the desire to use the Flickr module), so it's entirely possible that this is a case of user error as opposed to a bug. Nonetheless, I'm unable to get the Flickr filter to show pictures. The filter doesn't seem to touch the filter code at all! I've tried both Filtered HTML and Full HTML input formats, and nothing! I've tried both 5.x-0.0-BETA and HEAD versions.

Any advice appreciated! Also, as a side-note, does (or will) the Flickr module include LightBox or ThickBox support, or will that always be the domain of a separate module?

CommentFileSizeAuthor
#18 README_21.txt1.09 KBdrewish
#17 flickr-readme.txt1.17 KBsimplymenotu

Comments

andrewlevine’s picture

Have you entered your API Key and Secret at admin/settings/flickr?

As to lightbox/thickbox, I don't believe it's currently planned but patches implementing it would sure help boost its priority...

PilotChris’s picture

I do have the API key / secret entered. It took two tries, but the Flickr blocks do now work which I would think demonstrates that the key was provisioned successfully. It seems that the filter itself just isn't working, and I have no idea where to start debugging that.

andrewlevine’s picture

drewish knows a lot more about the filter than me but I can try.

What is the exact text you are entering into the body of the node? You aren't using any GUI editor like tinymce, ar eyou?

PilotChris’s picture

It seems that some other folks have had similar unresolved issues described here.

PilotChris’s picture

I basically copied the filter example from the Flickr module project page.

For example, I'm trying to embed this photo. I tried the filter text below:

[flickr-photo:id=470971384,size=s]

andrewlevine’s picture

I will look into this soon if drewish doesn't beat me to it

drewish’s picture

i just tried it out and it worked fine. did you add the filter to the input format you're trying to use (/admin/settings/filters)?

andrewlevine’s picture

Status: Active » Postponed (maintainer needs more info)

I reinstalled Drupal and tried as well. Worked fine. Let us know if drewish's comment in #7 works for you.

drewish’s picture

i think we ought to add a message in a hook_enable function for the the flickr_filter module to inform the user that they need to add the filter to the import format. some live documentation.

andrewlevine’s picture

I agree, although in the end we also need some proper documentation. I'll try to work a bit on this before our next release.

PilotChris’s picture

Thanks all. It was, in the end, user error. Drewish's suggestion in #7 (enabling the filter for the appropriate input formats) was what I had failed to do. Sorry to contribute to the noise level!

I'm circling at 10,000 feet, figuratively speaking, around trying to tackle a Thickbox integration for the Flickr module. I'm definitely not the "right" guy for the job in terms of Drupal module development expertise, except that I have a strong desire to see it working soon which seems to be a strong qualification in the open source world. ;)

drewish’s picture

Title: Flickr module filter does not work on new install! » Document installation of Flickr filter
Category: support » task
Status: Postponed (maintainer needs more info) » Active

PilotChris, glad you got it working. if you want to work on the lightbox stuff by all means, open up a new issue and start working on it. you'll be amazed at what you learn when you've got a project you want to figure out. and if it turns into something that's useful to others we can package it up as a sub-module.

I'm going to hijack this thread to remind me to add some help and a drupal_set_message when the module is enabled.

andrewlevine’s picture

PilotChris,

I've been thinking about adding thickbox integration for a while. It would be really useful if when you clicked an image instead of going to flickr.com a thickbox popped up with a larger image, comments, tags etc. I never had time to work on this and still don't really but I'd be happy to help if you wanted to bounce some ideas/questions/code off of me. Have fun...

drewish’s picture

keep in mind that flickr's TOS require that you use images hosted on their servers, you provide an obvious link to the original photo page. if you download a copy and serve that up, you don't need to provide the link.

eagereyes’s picture

++ on the documentation. I just tried out this module and one of the first threads I found was this one asking for help. I think you will get a lot more people to give you useful feedback with a little bit of documentation. A short README and a few lines in the help would be a huge step up from the current state.

drewish’s picture

I'm all for docs and I'd happily commit some but I've got a very finite amount of time I can spend on free Drupal stuff so docs end up getting shorted. So if any one wants to write up a README and attach it to this issue I'll spell check it and commit it.

simplymenotu’s picture

Component: flickr_filter » Documentation
StatusFileSize
new1.17 KB

README.txt attached for review and inclusion into this module.

drewish’s picture

Status: Active » Needs review
StatusFileSize
new1.09 KB

cool that looks good. i removed the maintainers section and changed some of the formatting around.

Anonymous’s picture

Status: Needs review » Reviewed & tested by the community

I'll get this committed , thanks

As this README.txt will need to be expanded to include new features ill add to the README that the instructions should be considered as unfinished/incomplete.

Best, Paul

Anonymous’s picture

Assigned: Unassigned »
Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)
lolandese’s picture

Issue summary: View changes

Documentation now on D.O.: https://drupal.org/node/2171503