Problem/Motivation

Drupal 7 allows you to add a configure line to a module's .info file to let users on the admin/modules page quickly jump to the module's settings.

It would be handy if the SugaronDrupal: Drupal connector for SugarCRM module did this.

Proposed resolution

Add the following line to sugarondrupal.info:

configure = admin/config/sugarondrupal

Remaining tasks

  1. Write a patch
  2. Review and RTBC
  3. Commit

User interface changes

On the module list page, a "configure" link will point to admin/config/sugarondrupal.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#7 Screen Shot 2017-04-07 at 7.36.04 PM.png32 KBkrina.addweb
#5 configure_link_is-2653702-5.patch418 bytesdhruveshdtripathi
#4 afterpatch-sugarCRM.PNG44.45 KBAanal.addweb
#3 configure_link_is-2653702-2.patch678 bytesmparker17
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mparker17 created an issue. See original summary.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 at The Linux Foundation commented
Assigned: mparker17 » Unassigned
Issue summary: View changes
Status: Active » Needs review

Here's a patch. Reviews welcome.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 at The Linux Foundation commented
FileSize
configure_link_is-2653702-2.patch678 bytes

I should probably attach that patch... :P

Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Needs work
FileSize
afterpatch-sugarCRM.PNG44.45 KB

@mparker17, Thanks for providing patch, but the configure link should redirect us on "https://dcmja.ply.st/admin/config/sugarondrupal/settings" instead of "https://dcmja.ply.st/admin/config/sugarondrupal" just like the configuration page flow. PFA

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
FileSize
configure_link_is-2653702-5.patch418 bytes

Thank you @dhwani.addweb for noticing the change. Changes made according to comment #4

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
Status: Needs work » Needs review
krina.addweb’s picture

krina.addweb CreditAttribution: krina.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
FileSize
Screen Shot 2017-04-07 at 7.36.04 PM.png32 KB

@dhruveshdtripathi, Thanks for providing updated patch, now it works well As I tested it using simplytest.me. PFA for current redirection link of configuration.

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
Issue summary: View changes