If you want to be able to export tac_lite scheme permissions using the structure_sync module, you might run into the problem, that the permission may different because the tid of taxonomy value has a different id on other systems. It would be good to have it use uuid instead so it's unique and also can be imported back into other environments.

I wrote a patch for that. Take a look at it and tell me if you like it.

Proposed solution

  • possibility to switch make a smooth transition from tid to uuid and back
  • performance improvements
CommentFileSizeAuthor
#4 3189168-tac_lite-use_uuid_for_schema_4.patch2.36 KBArnaud Kali
tac_lite-use_uuid_for_schema.patch2.53 KBrushw

Issue fork tac_lite-3189168

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

rushw created an issue. See original summary.

rushw’s picture

rushw commented
Issue summary: View changes

Status: Fixed » Closed (fixed)

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

Arnaud Kali’s picture

Arnaud Kali commented
Version: 8.x-1.5 » 8.x-1.6
StatusFileSize
new 3189168-tac_lite-use_uuid_for_schema_4.patch2.36 KB

Re-rolling patch onto latest version of module (1.6 at this time).

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Version: 8.x-1.6 » 2.0.x-dev
Assigned: rushw » Unassigned
Status: Closed (fixed) » Needs work

We are using this patch, but it was causing performance issues.
Reopening task for branch 2.0.
Few things to look into:

  • possibility to switch make a smooth transition from tid to uuid and back
  • performance improvements
vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Title: Make tac_lite scheme permissions exportable » Make tac_lite scheme work with tid or uuid
Issue summary: View changes

vladimiraus opened merge request !16

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Status: Needs work » Needs review
  • Introduced UUID / TID switch in configuration
  • Optimised processing logic
jannakha’s picture

jannakha commented
Status: Needs review » Reviewed & tested by the community

thank you for the fix!
feature works as required.

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Status: Reviewed & tested by the community » Fixed

Thanks for your contributions. Committed. 🍻

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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