Is it happening? What needs to be done?

CommentFileSizeAuthor
#10 flag_terms_d7.patch5.08 KBjide
#8 flag_terms.zip8.91 KBjide
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

KirstenLangholz’s picture

Also interested in it.

tayzlor’s picture

i'll take a look at the flag module d7 version and look at doing an upgrade, patches welcome!

rlmumford’s picture

this would be incredibly useful!

TimelessDomain’s picture

since Drupal 7 uses entities, wouldn't Flag module be able to work on Terms by default? If so then Flag Terms module won't need to be updated to D7

TimelessDomain’s picture

I have created such an issue in the Flag issue queue #1073568: Absorb Flag Terms into Flag Module

TimelessDomain’s picture

Patches have been submitted to absorb flag terms into flag module. So after these get committed we need to deprecate Flag Terms for D7 & make note of such on this projects homepage

tayzlor’s picture

@TimelessDomain - no prob i'll do such on the project page when this is the case.

jide’s picture

Status: Active » Needs review
Issue tags: +drupal 7
FileSize
8.91 KB

Hi all, comming from #1073568: Absorb Flag Terms into Flag Module.
To quote quicksketch from there :

Unless we get flagging terms for "free", I'm not really interested in integrating that module into Flag.

And moofie from #871064: Making flaggings fieldable :

(Flag 3.0 should be for D7 only. Flag 2.0 should not get any new features.)

So I guess a d7 port of this module is still needed. Here is my port attached.

kika’s picture

hi jide, can you provide the port as a patch?

jide’s picture

FileSize
5.08 KB

Hi kika, here is a patch.

CUclimber’s picture

Adding my interest to this. This would be incredibly useful for me on my D7 site. Thanks!

achton’s picture

Subscribe. This is way cool - why doesn't anyone use it more? E.g. for a gaming website, I would be doing a "I like this gaming genre"-flag :-)

alexbk66-’s picture

+1

TimelessDomain’s picture

Status: Needs review » Reviewed & tested by the community

This works! Thanks. I just pulled it up in views. The only setting that did not work is "Display link on taxonomy term pages," which is pretty easy to solve otherwise

fangel’s picture

It does indeed works great. However, it would be a bit better if it saved the flaggable taxonomies by machine-name, not vocab-id. When combined with features, deployment can change which terms are flaggable if the vocabs have different ids across servers.

Ryan Chu’s picture

it'll be very useful for me, hope this module be ported to D7~

kostajh’s picture

Could we get a 7.x branch opened up based on the work in #10?

grah’s picture

+1

rogical’s picture

+1

mefisto75’s picture

agree with kostajh. Project page may have to be updated.

tayzlor’s picture

I'm on this - will open up a 7.x branch in the next few days

Ryan Chu’s picture

GREAT!
Thank you, @tayzlor.
Great module! Thanks you!

Liliplanet’s picture

subscribe, thx!

mototribe’s picture

+1

tayzlor’s picture

sorry for the delay guys, should now be a snapshot branch up on the project page!

tayzlor’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

Everett Zufelt’s picture

Version: 6.x-1.0-beta2 » 7.x-1.x-dev

Switching version so that this issue can be more easily discovered.