Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

Hides language tabs from regional admins#5167

Merged
itsjoekent merged 5 commits intoDoSomethingArchive:devfrom
itsjoekent:5113
Sep 22, 2015
Merged

Hides language tabs from regional admins#5167
itsjoekent merged 5 commits intoDoSomethingArchive:devfrom
itsjoekent:5113

Conversation

@itsjoekent
Copy link
Contributor

What's this PR do?

Hides the language tabs at the top of /node/edit/ from regional admins

How should this be manually tested?

Edit a node as a regional admin and verify the tabs aren't there.

Any background context you want to provide?

Looked in both menu_alter $items and the node_form alter $form and couldnt find them set in either. Figured this was an easier solution.

What are the relevant tickets?

Fixes #5113

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

meh, this isn't really best practice. this should be available in hook_menu_local_tasks_alter()
if you're having trouble seeing it there, you might need to play with the module weight

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ohhh local_tasks_alter -- I was doing menu_alter

Ill try that one!

@itsjoekent
Copy link
Contributor Author

@angaither updated to be more :drupal-y:

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

commmmments

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💬

@itsjoekent
Copy link
Contributor Author

@angaither added commmennnttsssss 💬

@angaither
Copy link
Contributor

👍

itsjoekent pushed a commit that referenced this pull request Sep 22, 2015
Hides language tabs from regional admins
@itsjoekent itsjoekent merged commit b584c74 into DoSomethingArchive:dev Sep 22, 2015
@itsjoekent itsjoekent deleted the 5113 branch September 22, 2015 14:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants