Move configuration menu item to "Regional and language" [#1272306]

Can the following change please be made to version 7:

$items['admin/config/gtranslate'] = array(
        'title' => 'GTranslate',
        'description' => 'GTranslate module settings control',
        'page callback' => 'drupal_get_form',
        'page arguments' => array('gtranslate_admin'),
        'access arguments' => array('administer site configuration'),
        'type' => MENU_NORMAL_ITEM
    );

Changes the menu path and the permission. This puts Gtranslate in the right spot under /config (not in the Toolbar) with a more appropriate permission.

Comment	File	Size	Author
#13	Screenshot 2023-02-23 at 11.29.28.png	59.31 KB	i-trokhanenko
#10	1272306-fix-gtranslate-menu.patch	486 bytes	SocialNicheGuru
#8	gtranslate-fix_config_menu_and_access-1272306-8.patch	730 bytes	mcannon
#3	1272306-1.patch	680 bytes	beltofte

Issue fork gtranslate-1272306

Show commands

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

Add & fetch this issue fork’s repository

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

Add & fetch this issue fork’s repository

1272306-move-configuration-menu changes, plain diff MR !10
Check out this branch

About issue forks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

kevinquillen CreditAttribution: kevinquillen commented 7 September 2011 at 18:11

Sorry, that menu path should be 'admin/config/regional/gtranslate'.

Comment #2

beltofte

Danish

Copenhagen 🇩🇰

CreditAttribution: beltofte commented 24 September 2012 at 11:51

Title:

Minor fixes

» Move configuration menu item to "Regional and language"

Comment #3

beltofte

Danish

Copenhagen 🇩🇰

CreditAttribution: beltofte commented 24 September 2012 at 12:17

File	Size
1272306-1.patch	680 bytes

The attached patch fixes this issue. The module it self could really need heave code clean up to follow the official Drupal coding standards. Would gladly do this if the maintainer was not selling commercial versions of the code!

Comment #4

kevinquillen CreditAttribution: kevinquillen commented 24 September 2012 at 12:30

Aren't there a lot of new Google APIs out now? Why not branch off a new ver? You can request maintainer status from d.o. if the original maintainer is not around.

Comment #5

beltofte

Danish

Copenhagen 🇩🇰

CreditAttribution: beltofte commented 25 September 2012 at 08:13

Not planning to be part of this project and maintain it, because the current maintainer is distribution and developing on http://gtranslate.net instead of d.o, and also selling commercial versions of it http://gtranslate.net/features.

So if I need a module like this, would I probably create a new module and project on d.o.

Comment #6

Simon Georges CreditAttribution: Simon Georges commented 26 August 2013 at 14:15

Status:

Active

» Reviewed & tested by the community

I can confirm this patch works as expected, and is needed to have the configuration screen correctly working.

Comment #7

Simon Georges CreditAttribution: Simon Georges commented 26 August 2013 at 14:15

Version:

7.x-1.7

» 7.x-1.9

In current version too.

Comment #8

mcannon CreditAttribution: mcannon commented 25 April 2014 at 14:11

Issue summary:	View changes
Status:	Reviewed & tested by the community	» Needs review

File	Size
gtranslate-fix_config_menu_and_access-1272306-8.patch	730 bytes

I also noticed this issue along with a permission issue of "access administration pages" which is a perm we give most all of our clients so their editing experience is the same on every site. So this made GTranslate settings accessible to editors when it should not be.

The patch I attached solves the path and the access.