Download & Extend
Taxonomy Access Control » Issues

TAC: D7 port

Posted by xjm on November 4, 2009 at 9:40pm
Project:Taxonomy Access Control
Version:7.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

I'm just wondering if you're planning for a D7 version of TAC, and if so, what sort of a timeline it's on.

Login or register to post comments

Comments

#1

Posted by xjm on February 21, 2010 at 3:41am
Status:active» postponed

#2

Posted by xjm on June 7, 2010 at 4:23pm

Marked #820400: Drupal 7 version as duplicate of this issue.

There's no D7 branch at present. I've postponed this issue since I don't have time this summer to work on a D7 port of the module. I'd like to see a D7 version, but I'm not going to be maintaining any D7 sites during the next fiscal year. I don't think keve has plans to work on a port either.

If someone else creates a port I can help with its development this fall, but it's unlikely it will make D7 release.

#3

Posted by xjm on August 2, 2010 at 2:06pm

See also: #871572: Combine forces: merge TAC and TAC_Lite into one package in D7?

#4

Posted by LPCA on September 11, 2010 at 6:23am

subscribing

#5

Posted by jppi_Stu on October 18, 2010 at 10:52pm

This is a critical part of some Drupal systems I manage, so I'll definitely be keeping an eye on this. Without looking at the underlying code or comparing functions, at a basic level it seems sensible to pursue a TAC/TAC_Lite merge.

#6

Posted by servantleader on December 4, 2010 at 7:57pm

Subscribe

#7

Posted by binford2k on December 14, 2010 at 2:06am
Status:postponed» active

subscribe

#8

Posted by jspotswood on January 6, 2011 at 1:12am

subscribe

#9

Posted by MrDamasajaga on January 8, 2011 at 1:09pm

subscribe
#3 interesting, Would love a more lightweight version and reducing duplicate functionability

#10

Posted by mdijkstr on January 12, 2011 at 1:00pm

subscribe

#11

Posted by kultmarke on January 12, 2011 at 1:38pm

subscribe

#12

Posted by sw3b on January 18, 2011 at 8:09pm

subscribe

It would be nice to have it in D7. Already using it in D6 and will have to wait for something similar before moving to D7.

#13

Posted by Anticosti on February 1, 2011 at 2:57pm

subscribing

#14

Posted by dantz on February 6, 2011 at 9:19pm

subscribe

#15

Posted by ssace on February 11, 2011 at 12:29am

subscribe

#16

Posted by saidev on February 20, 2011 at 7:41am

subscribe

#17

Posted by FrequenceBanane on February 25, 2011 at 4:39pm

Hello !
#623660-3: TAC: D7 port#3 seems to me a great idea !
Do you already have a porting plan, or decided what would be in the liteweight version of TAC_Lite/TAC and what what would maybe come as a complementary module to add the other features TAC and TAC_Lite currently offers?
I don't know much of these modules, but I have some time to code (with let's say a guiding help from someone who know where this module combinaison is going ...)...

#18

Posted by Goulou on February 23, 2011 at 8:59am

subscribe

#19

Posted by xjm on February 25, 2011 at 5:33pm

@ #17: I haven't started work on this yet, but I'm hoping to do some planning two weeks from now at DrupalCon. I'm still interested in making TAC and TAC_Lite one project, but TAC_Lite opened a partial port to D7 just yesterday.

Some more information for those who are curious. TAC by itself is going to be less than straightforward to port, for three reasons:

  1. Database abstraction layer changes: DBTNG has a big impact on TAC, because we use a BIT_OR aggregate that is not exactly your run-of-the-mill SQL. Historically even older postgres has required special support within the module.
  2. Taxonomy changes: The taxonomy system has been entirely redone in D7 and there are lots of new hooks.
  3. Core access control changes: There are lots of new hooks available and we may want to take advantage of some of these.

#20

Posted by Danny_Joris on March 1, 2011 at 9:53pm

subscribing

#21

Posted by Encarte on March 2, 2011 at 7:14pm

subscribe

#22

Posted by tmuras on March 2, 2011 at 9:38pm

Hi All,

I agree with xjm. I'm planning to create something similar to acl module, that can be a base for any taxonomy-based frontend. This will have all the features of both tac and tac_lite. In the longer term, I think it should be possible to merge the module with acl.

Please contact me, if you're interested in cooperation. The more the merrier!

Tomek

#23

Posted by xjm on March 6, 2011 at 3:51am

@tmuras / anyone else who's interested: If you're going to be at DrupalCon, let me know if you're interested in talking about TAC in D7 or helping port it. (You can use my contact form to email me.)

#24

Posted by Dru-p on March 8, 2011 at 12:04am
Title:D7 port?» TAC: D7 port

Following.

#25

Posted by xjm on July 5, 2011 at 9:41am

Here's where I am in the D7 port process. I'll update this as I work through it. Update: the 7.x-1.x branch is now open and a -dev snapshot will be available soon. However, the branch is more or less nonfunctional at this point, so don't get your hopes up just yet. :) Please let me know if you want to help work on one of the tasks below.

  1. Run through coder for obvious API changes. Done
  2. Get V/U/D ops in hook_node_access_records() working properly for automatically-generated term access data. Done
  3. #1097782: Admin forms in D7 Done
  4. Research DBTNG API and migrate queries. Done
  5. #1097770: Update schema for D7
  6. #1097776: Update hook_menu() for D7
  7. #1097798: D7: implement create and list ops and provide API
  8. #1097804: TAC Fields D7 port
  9. #1097806: Provide Views 7.x-3.x filters for each permission
  10. #1097816: Update INSTALL.txt, UPDATE.txt, and README.txt

#26

Posted by xjm on March 14, 2011 at 8:19am
Version:master» 7.x-1.x-dev

#27

Posted by xjm on March 14, 2011 at 8:19am
Status:active» needs work

#28

Posted by David_Rothstein on March 18, 2011 at 1:25pm
Category:support request» task

Hey, I'm interested in helping out a bit. (After too much time working on D7 core and with that finally released, I've recently started to remember again that contrib modules exist and have been trying to work on some of them, so you caught me at a good time at DrupalCon :)

I can volunteer to work on items #4, #5, and #6; they seem like the best places to jump in. I hope to get to this in the next several days.

Since there is now a 7.x-1.x branch, does it make sense to close this issue and open up smaller ones for the actual tasks that need to be done?

#29

Posted by xjm on March 18, 2011 at 5:54pm

Thanks David! I opened the following issues:

#1097770: Update schema for D7
#1097776: Update hook_menu() for D7
#1097782: Admin forms in D7
#1097798: D7: implement create and list ops and provide API

I'll leave this issue open for now so I can reference it on the project page.

#30

Posted by xjm on March 18, 2011 at 6:11pm

#1097804: TAC Fields D7 port
#1097806: Provide Views 7.x-3.x filters for each permission
#1097816: Update INSTALL.txt, UPDATE.txt, and README.txt

#31

Posted by shadowmihai on March 25, 2011 at 12:05am

subscribe

#32

Posted by krueschi on March 29, 2011 at 1:13pm

subscribing

#33

Posted by jan.koprowski on March 30, 2011 at 4:38am

subscribing

#34

Posted by anthonyR on April 1, 2011 at 8:26am

subscribe

#35

Posted by Keiichi on April 3, 2011 at 10:13pm

subscribe

#36

Posted by hclaus on April 7, 2011 at 7:23pm

subscribe

#37

Posted by Osiris Indriya on April 10, 2011 at 5:00am

subscribe

#38

Posted by spam38 on April 20, 2011 at 4:53pm

As part of my employer's drupal efforts, I have written a Drupal 7 version of TAC from scratch. If anybody else would like to try it out, its available from svn at http://gforge.unl.edu/svn/wdn_thm_drupal/branches/drupal-7.x/staging/sit... (username anonymous, no password)

#39

Posted by shadowmihai on April 20, 2011 at 9:10pm

@ spam38 - your link, after logging in, gives me an error message - "no mime type"

#40

Posted by xjm on April 22, 2011 at 9:52am

Hmmm. If the port works, that's awesome, but I'm a bit concerned about it being written from scratch. It would be missing seven years of bugfixes, if that makes sense.

In any case, quite interested to see the code, if a working archive can be posted. I've had to put my Drupal development on hold for a few weeks because of other requirements at work, but I will take a look at it whenever I get back to working on the port.

By the way, if anyone else wants to help with the existing port, there is a D7 branch in git and there are several specific issues open (as above).

#41

Posted by Leeteq on June 1, 2011 at 4:59am

Subscribing.
Edit: FYI: 7.x-1.x-dev: http://drupal.org/node/1089904
(not showing on the module front page, but on the "all releases" page)

#42

Posted by tonycpsu on April 29, 2011 at 8:12pm

Subscribing

#43

Posted by mhahndl on May 5, 2011 at 10:04am

Subscribing

#44

Posted by luckystrikerin on May 11, 2011 at 11:13am

subs

#45

Posted by alexbern on May 12, 2011 at 7:29am

subscribing

#46

Posted by sheila8stamp on May 15, 2011 at 4:19pm

subscribing

#47

Posted by joostvdl on May 16, 2011 at 2:13pm

subscribe

#48

Posted by paulgemini on May 17, 2011 at 4:22am

subscribe

#49

Posted by bjcool on May 20, 2011 at 10:33am

subscribing

#50

Posted by FireMouse on May 31, 2011 at 7:17pm

Subscribing.

#51

Posted by xjm on June 8, 2011 at 1:16pm

Re: #41 -- That is because it is not functional yet. :) I unfortunately had to drop work on this due to other circumstances.

#52

Posted by marvix on June 11, 2011 at 9:20am

subscribe

#53

Posted by xjm on June 20, 2011 at 1:01pm

Just to let everyone know, the Austrian BIFIE is sponsoring development of the 7.x-dev branch. A functional development version should be available by July 11.

#54

Posted by rafaeld on June 21, 2011 at 5:45am

subscribe

#55

Posted by Bitnetix on June 22, 2011 at 7:21pm

me too

#56

Posted by az100sun on June 23, 2011 at 1:21pm

Happy to test and provide feedback when the dev version becomes available.

#57

Posted by xjm on June 25, 2011 at 12:13am

I have added the issue tag d7 release for issues that need to be resolved prior to a D7 release. The next major task is to get the create and list ops working (#1097798: D7: implement create and list ops and provide API). After that, the dev branch should be functional enough for people to begin testing.

#58

Posted by modstore on July 3, 2011 at 10:48pm

subscribe

#59

Posted by xjm on July 5, 2011 at 1:36am
Status:needs work» fixed

Drupal 7 Port Complete

The Drupal 7 port is complete, and the development build is available for testing:
http://drupal.org/node/1089904

Notes and Known Issues:

  1. All the module's basic functionality is present. View, Update, Delete, Create, and List grants are functional; and access is updated in response to node and taxonomy changes.
  2. The module's internal API has been refactored significantly, improving perfomance, permitting better interoperability, and reducing the risk of future regressions.
  3. The create and list grants have been entirely re-designed in the backend, in part because of API changes and in part because of many previous and outstanding bugs in 6.x-1.x and earlier branches:

Recommendations

  1. Do not deploy the dev build in production without significant testing.
  2. Use the Devel module during testing for accurate feedback on node access behaviors. For more information on using devel, see the module documentation. (There may be minor inconsistencies with Drupal 7 in the instructions.)
  3. If you are using a field widget other than those provided by core, Multiselect, or Hierarchical Select, or if you are using some type of taxonomy field other than core's term reference field, test to see if the field behaves as expected when the user does not have create grants. If it doesn't, please open a feature request to support this widget.

What is needed for a 7.x-1.0 release

  1. Help test! I do not have any production Drupal 7 sites, so I need as much help testing the module as possible.
  2. Issues tagged d7 release need to be resolved before we create a release version.
  3. Please, please report any bugs you find!

If the 7.x-1.x-dev build of module works for you, feel free to leave a comment below! However, if you encounter a bug or have a feature request, please do not re-open this issue. Instead, check the open 7.x issues or create a new issue if yours is not yet listed.

#60

Posted by System Message on July 19, 2011 at 9:51am
Status:fixed» closed (fixed)

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