Internationalize all strings in admin_modules.hmtl and corrected comments.

2 files changed, 7 insertions, 5 deletions

diff --git a/modules/gallery/helpers/module.php b/modules/gallery/helpers/module.php
index 2ae83f0d..595f600b 100644
--- a/modules/gallery/helpers/module.php
+++ b/modules/gallery/helpers/module.php
@@ -122,6 +122,7 @@ class module_Core {
   /**
    * Check that the module can be installed. (i.e. all the prerequistes exist)
    * @param string $module_name
+   * @return array an array of warning or error messages to be displayed
    */
   static function check_environment($module_name) {
     module::_add_to_path($module_name);
@@ -132,14 +133,15 @@ class module_Core {
       $messages = call_user_func(array($installer_class, "check_environment"));
     }
 
-    // Now the module is installed but inactive, so don't leave it in the active path
+    // Remove it from the active path
     module::_remove_from_path($module_name);
     return $messages;
   }
 
   /**
-   * Check that the module can be installed. (i.e. all the prerequistes exist)
+   * Allow modules to indicate the impact of deactivating the specifeid module
    * @param string $module_name
+   * @return array an array of warning or error messages to be displayed
    */
   static function can_deactivate($module_name) {
     $data = (object)array("module" => $module_name, "messages" => array());
diff --git a/modules/gallery/views/admin_modules.html.php b/modules/gallery/views/admin_modules.html.php
index 7f572411..704e7beb 100644
--- a/modules/gallery/views/admin_modules.html.php
+++ b/modules/gallery/views/admin_modules.html.php
@@ -18,12 +18,12 @@
             height: 400,
             width: 500,
             position: "center",
-            title: "Confirm Module Activation",
+            title: <?= t("Confirm Module Activation")->for_js() ?>,
             buttons: {
-              "Continue": function() {
+              <?= t("Continue")->for_js() ?>: function() {
                 $("form", this).submit();
               },
-              Cancel: function() {
+              <?= t("Cancel")->for_js() ?>: function() {
                 $(this).dialog("destroy").remove();
               }
             }