I'm sorry, but a lot of the text on the settings page wasn't very clear. The attached patch rewords the UI text and also moves the admin form into its own separate file, drupalgapps.admin.inc, which is a bit more standard. By having it in a separate file, that code is only loaded when it is needed and helps improve the site performance ever so slightly, especially on the modules listing page. I also added doxygen style comments to group functions logically together.

Dependant on #685606: coding style cleanup and #685670: modularlize the code being applied first.

Cheers,
Stella

CommentFileSizeAuthor
#2 685782.patch31.61 KBstella
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SocialNicheGuru’s picture

Version: » 6.x-1.0-beta1

was there a patch here.

stella’s picture

FileSize
31.61 KB

Doh! Here it is now. Sorry about that.

stella’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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