Migration to Drupal 7

Hi Paul and others,

I finished the conversion of the base module to Drupal 7. I implemented all new hooks and definitions. The only thing I couldn't convert is the hook_user_update (now called flickr_user_presave). The function db_affected_rows() doesn't exist anymore, so we should implement another way of checking if a user has been deleted. (#509122: RIP db_affected_rows())

That said, all is working correctly. See the attachment.

Now I will start working on the flickr_block module first. Can you provide me with CVS access so I can commit my code?
I'd like to offer to help you with the D6 issue queue too.

Forgot the CVS header at the CSS file.

V3, now with flickr_block integration. The rest will follow after I got CVS access :)

Hi all,

Here is a new version.

- I corrected a small error in the flickr_block module (theme function attributes, photo instead of p)
- I added the flickr_filter module. Now also works in Drupal 7! :)

I take the module one at a time. Only left now are the flickr_field, flickr_tags and flickr_sets modules. Working on those!

You are welcome! You take a lot and you give back some, isn't that the idea of the community?

By the way, you can see the Block module working on my Drupal blog already. :D

Hi Paul,

I'll include his patch, no problem!
I'm currently working on the Tags and Sets modules.

They are working now in Drupal 7, but they are not doing what I expect them to do.
The problem is that they are not working as expected in the Drupal 6 version as well.

It is not clear to users that the url flickr/% exists. This is not explained in the README.TXT and the menu system doesn't link to it. My expectations were to find the Flickr listed in the user tabs. For instance, on user/1 we could have a tab linked to user/1/flickr. And also user/1/flickr/sets and user/1/flickr/tags.

I'll leave this for now and will port the module as is, with the same functionality.
We'll start another issue for the menu paths.

Flickr_sets and Flickr_tags included. Only left now is the Flickrfield module.
That one is a bit harder because CCK has been replaced by fields, which has caused some functions to be renamed and refactored.

Working on that one!

Hi Paul,

thanks for testing. I fixed both issues.
The first error you gave was simple: I overlooked the user_load() change.
The other error was a bit more difficult. The problem with the current setup is that we have the following urls:

/flickr/1/tags/list
/flickr/1/tags/cloud
/flickr/1/tags/amsterdam

In the first two examples, the part after /tags/ is the function, but in other cases, this is an argument. Drupal cannot know if 'list' is an argument or a function call. For example: what if the tag is 'cloud'?

Therefore I made a change in the functionality. I used the old urls /flickr/1/tags/list and /flickr/1/tags/cloud for function calls and for the argument calls I use /flickr/1/tag/ARGUMENT (so tag instead of tags).

What would you think of this solution?

Hi Paul,

just to make sure: I changed the path to the Flickr tags sets.
Instead of /flickr/1/tags/Amsterdam the link should be /flickr/1/tag/Amsterdam.

We need to use a workaround to make sure Drupal does not see /list and /cloud as tags.

Thanks for the credits on the project page ;)

Paul,

Still working on the Fields migration. I'm almost done, but currently I'm spending most of my time working on my Google Fonts module.

Next weekend I'll try to finish the D7 version of the Flickr module!

Hi Paul,

only the Flickr Fields module needs to be ported, the rest is done. I tried and failed so far, the field API is just way over my head.
If anyone wants to jump in, please feel free to do so.