Is it happening?

CommentFileSizeAuthor
#23 views_flag_refresh_d7_v1.1.zip13.61 KBnquocbao

Comments

Shadlington’s picture

Title: Drupal 7 port » Drupal 7 port: Views Flag Refresh

Subbing - very interested in seeing a D7 version!

dgastudio’s picture

sub!

cpliakas’s picture

Hey guys. Most of my development effort is being spent on Facet API, so I won't have time to attend to this module in the near future. Would love to get some help with this and would be more than willing to pick up a co-maintainer. I am also open to starting a sandbox project and giving access to whomever so we can facilitate the update.

Thanks,
Chris

rv0’s picture

sub

walker2238’s picture

Subscribe. Love this module.

rv0’s picture

eventually i needed a quick fix, so I borrowed some code from http://drupal.org/project/block_refresh and wrote some jquery to handle the flagging event. working fine.
wish i had some more time to devote to this.

wolfmarter’s picture

subscirbing

i would also really like to see a d7 port.

@rv0 how did you get this done?

rv0’s picture

@wolfmarter - as i said, by borrowing code from the block_refresh project which has a D7 port... My solution isn't clean nor generic.

wolfmarter’s picture

my problem is that the refresh block module removes the javascript in the url of the remove button

i am trying to fix this but with no luck so far

help appreciated

thx

wolfmarter

murz’s picture

subscirbing

webmasterkai’s picture

Subscribing. I might take a stab at creating a d7 port .

cpliakas’s picture

webmasterkai Excellent! Thanks in advance.

cm_is’s picture

subscribing

webadpro’s picture

I would really enjoy having a port for this also.

rodych’s picture

+

Anonymous’s picture

subscribing.

I need to figure this out, so for now I'll look at using the block_refresh module, but I may just write something custom.

cm_is’s picture

any update on this one? are you working on this? regards cm

webmasterkai’s picture

Yes, this is being worked on. We should have a patch in the next couple days.

cm_is’s picture

Great to hear! really looking forward to the patch!

webadpro’s picture

Thats great news!

Anonymous’s picture

Any news on this issue?

thehong’s picture

The team finished this port, nqbao will attach the patch soon.

nquocbao’s picture

StatusFileSize
new13.61 KB

Here is the D7 port for views_flag_refresh. Looking forward to any feedback.

Cheers

cpliakas’s picture

Excellent work! If others can RTBC this, do you want commit access?

Thanks,
Chris

nquocbao’s picture

Yes, it would be great. We can create a 7.x-1.x branch then commit the patch there. Most changes are in views_flag_refresh.js

cpliakas’s picture

Hi nquocbao.

I created the 6.x-1.x and 7.x-1.x branches respectively and gave you commit access. I also gave you the ability to roll releases, so do so at your will.

Thanks!
Chris

bjalford’s picture

Status: Active » Reviewed & tested by the community

Works fine for me

nquocbao’s picture

Hi all,

I rolled out a 7.x-1.0-beta1. http://drupal.org/node/897498/release

Cheers :)

webmasterkai’s picture

Status: Reviewed & tested by the community » Fixed
cpliakas’s picture

Thanks nquocbao.

Just an FYI I didn't merge your changes into the 7.x-1.x branch. Sorry for not being clear on this. Did you get a chance to merge your stuff in prior to cutting the tag?

Thanks,
Chris

nquocbao’s picture

Hi cpliakas,

I merged my changes before tagging it. Here is the commit: http://drupal.org/commitlog/commit/15578/41e5e026cd621c3b171c3a1292f61a5...

cpliakas’s picture

Great! Your commit doesn't show up in the "Maintainers for Views Flag Refresh" block nor in the "View all committers" list which is why I was confused. Thanks for pointing out the commit, and this seems like a D.O. bug that should be fixed. Your name should be there!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.