Fixing lots of warnings and code style errors detected by Moodle's bot.

Author: Ravenink

Diff for: amd/src/buttons_states.js

@@ -1,49 +1,49 @@
-// This file is part of the Moodle module "EJSApp"
+// This file is part of the Moodle block "EJSApp File Browser"
 //
-// EJSApp is free software: you can redistribute it and/or modify
+// EJSApp File Browser is free software: you can redistribute it and/or modify
 // it under the terms of the GNU General Public License as published by
-// the Free Software Foundation, either moodle_version 3 of the License, or
-// (at your option) any later moodle_version.
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
 //
-// EJSApp is distributed in the hope that it will be useful,
+// EJSApp File Browser is distributed in the hope that it will be useful,
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 // GNU General Public License for more details.
 //
-// The GNU General Public License is available on <http://www.gnu.org/licenses/>
+// You should have received a copy of the GNU General Public License
+// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
 //
-// EJSApp has been developed by:
-//  - Luis de la Torre: [email protected]
-//	- Ruben Heradio: [email protected]
+// EJSApp File Browser has been developed by:
+// - Luis de la Torre: [email protected]
+// - Ruben Heradio: [email protected]
 //
-//  at the Computer Science and Automatic Control, Spanish Open University
-//  (UNED), Madrid, Spain
+// at the Computer Science and Automatic Control, Spanish Open University
+// (UNED), Madrid, Spain.
 
 /**
  * Actions associated to the buttons "Show" and "Hide" of the block.
  *
- * @package    block
- * @subpackage ejsapp_file_browser
+ * @package    block_ejsapp_file_browser
  * @copyright  2016 Luis de la Torre and Ruben Heradio
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
 define(['jquery'], function($) {
     var t = {
-        init : function(reproducing) {
-            $("#hide_blockly").click(function(){
+        init: function(reproducing) {
+            $("#hide_blockly").click(function() {
                 $("#blocklyControl").hide();
             });
 
-            $("#show_blockly").click(function(){
+            $("#show_blockly").click(function() {
                 $("#blocklyControl").show();
             });
 
-            $("#hide_interaction").click(function(){
+            $("#hide_interaction").click(function() {
                 $("#captureInteraction").hide();
             });
 
-            $("#show_interaction").click(function(){
+            $("#show_interaction").click(function() {
                 $("#captureInteraction").show();
             });
 
@@ -56,28 +56,28 @@ define(['jquery'], function($) {
                 $("#playCaptureBut").prop('disabled', true);
             }
 
-            $("#startCaptureBut").click(function(){
+            $("#startCaptureBut").click(function() {
                 $("#startCaptureBut").prop('disabled', true);
                 $("#stopCaptureBut").prop('disabled', false);
                 $("#resetCaptureBut").prop('disabled', false);
                 $("#playCaptureBut").prop('disabled', true);
                 $("#stepCaptureBut").prop('disabled', true);
             });
 
-            $("#stopCaptureBut").click(function(){
+            $("#stopCaptureBut").click(function() {
                 $("#startCaptureBut").prop('disabled', false);
                 $("#stopCaptureBut").prop('disabled', true);
                 $("#resetCaptureBut").prop('disabled', false);
             });
 
-            $("#resetCaptureBut").click(function(){
+            $("#resetCaptureBut").click(function() {
                 $("#startCaptureBut").prop('disabled', false);
                 $("#stopCaptureBut").prop('disabled', true);
                 $("#resetCaptureBut").prop('disabled', true);
                 $("#playCaptureBut").prop('disabled', false);
             });
 
-            $("#playCaptureBut").click(function(){
+            $("#playCaptureBut").click(function() {
                 $("#startCaptureBut").prop('disabled', true);
                 $("#stopCaptureBut").prop('disabled', true);
                 $("#resetCaptureBut").prop('disabled', false);

Diff for: block_ejsapp_file_browser.php

@@ -1,24 +1,30 @@
 <?php
-// This file is part of Moodle - http://moodle.org/
+// This file is part of the Moodle block "EJSApp File Browser"
 //
-// Moodle is free software: you can redistribute it and/or modify
+// EJSApp File Browser is free software: you can redistribute it and/or modify
 // it under the terms of the GNU General Public License as published by
 // the Free Software Foundation, either version 3 of the License, or
 // (at your option) any later version.
 //
-// Moodle is distributed in the hope that it will be useful,
+// EJSApp File Browser is distributed in the hope that it will be useful,
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 // GNU General Public License for more details.
 //
 // You should have received a copy of the GNU General Public License
-// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
+//
+// EJSApp File Browser has been developed by:
+// - Luis de la Torre: [email protected]
+// - Ruben Heradio: [email protected]
+//
+// at the Computer Science and Automatic Control, Spanish Open University
+// (UNED), Madrid, Spain.
 
 /**
  * Private files block caps.
  *
- * @package    block
- * @subpackage ejsapp_file_browser
+ * @package    block_ejsapp_file_browser
  * @copyright  2012 Luis de la Torre and Ruben Heradio
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */

Diff for: db/access.php

@@ -1,5 +1,4 @@
 <?php
-
 // This file is part of the Moodle block "EJSApp File Browser"
 //
 // EJSApp File Browser is free software: you can redistribute it and/or modify
@@ -12,25 +11,24 @@
 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 // GNU General Public License for more details.
 //
-// The GNU General Public License is available on <http://www.gnu.org/licenses/>
+// You should have received a copy of the GNU General Public License
+// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
 //
 // EJSApp File Browser has been developed by:
-//  - Luis de la Torre: [email protected]
-//	- Ruben Heradio: [email protected]
+// - Luis de la Torre: [email protected]
+// - Ruben Heradio: [email protected]
 //
-//  at the Computer Science and Automatic Control, Spanish Open University
-//  (UNED), Madrid, Spain
-
+// at the Computer Science and Automatic Control, Spanish Open University
+// (UNED), Madrid, Spain.
 
 /**
  * Manage files in folder in private area
- *     
- * @package    block
- * @subpackage ejsapp_file_browser
+ *
+ * @package    block_ejsapp_file_browser
  * @copyright  2012 Luis de la Torre and Ruben Heradio
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
 require('../../config.php');
-
+require_login(0, false);
 redirect(new moodle_url('/user/files.php'));

Diff for: edit.php

@@ -11,25 +11,24 @@
 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 // GNU General Public License for more details.
 //
-// The GNU General Public License is available on <http://www.gnu.org/licenses/>
+// You should have received a copy of the GNU General Public License
+// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
 //
 // EJSApp File Browser has been developed by:
-//  - Luis de la Torre: [email protected]
-//	- Ruben Heradio: [email protected]
+// - Luis de la Torre: [email protected]
+// - Ruben Heradio: [email protected]
 //
-//  at the Computer Science and Automatic Control, Spanish Open University
-//  (UNED), Madrid, Spain
-
+// at the Computer Science and Automatic Control, Spanish Open University
+// (UNED), Madrid, Spain.
 
 /**
  * English labels for the ejsapp_file_browser block
- *    
- * @package    block
- * @subpackage ejsapp_file_browser
+ *
+ * @package    block_ejsapp_file_browser
  * @copyright  2012 Luis de la Torre and Ruben Heradio
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
- 
+
 $string['title_of_the_block'] = 'EJSApp File Browser';
 $string['managemyfiles'] = 'Manage my files';
 $string['pluginname'] = 'EJSApp "private files" browser';
@@ -51,11 +50,11 @@
 $string['play_capture'] = 'Load';
 $string['change_speed'] = 'Velocity:';
 
-// settings.php:
+// Strings in settings.php.
 $string['auto_refresh_header_config'] = 'Configure the block\'s auto-refresh property';
 $string['auto_refresh'] = 'Auto-refresh frequency';
 $string['auto_refresh_description'] = 'Time in miliseconds. Write "0" to disable auto-refresh.';
 
-//Capabilities
+// Strings for capabilities.
 $string['ejsapp_file_browser:addinstance'] = 'Add a new private files block for EJSApp';
 $string['ejsapp_file_browser:myaddinstance'] = 'Add a new private files block for EJSApp to My home';

Diff for: lang/en/block_ejsapp_file_browser.php

@@ -1,5 +1,4 @@
 <?php
-
 // This file is part of the Moodle block "EJSApp File Browser"
 //
 // EJSApp File Browser is free software: you can redistribute it and/or modify
@@ -12,21 +11,20 @@
 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 // GNU General Public License for more details.
 //
-// The GNU General Public License is available on <http://www.gnu.org/licenses/>
+// You should have received a copy of the GNU General Public License
+// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
 //
 // EJSApp File Browser has been developed by:
-//  - Luis de la Torre: [email protected]
-//	- Ruben Heradio: [email protected]
+// - Luis de la Torre: [email protected]
+// - Ruben Heradio: [email protected]
 //
-//  at the Computer Science and Automatic Control, Spanish Open University
-//  (UNED), Madrid, Spain
-
+// at the Computer Science and Automatic Control, Spanish Open University
+// (UNED), Madrid, Spain.
 
 /**
  * Spanish labels for the ejsapp_file_browser block
- *      
- * @package    block
- * @subpackage ejsapp_file_browser
+ *
+ * @package    block_ejsapp_file_browser
  * @copyright  2012 Luis de la Torre and Ruben Heradio
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
@@ -52,11 +50,11 @@
 $string['play_capture'] = 'Cargar';
 $string['change_speed'] = 'Velocidad:';
 
-// settings.php:
+// Strings in settings.php.
 $string['auto_refresh_header_config'] = 'Configurar la propiedad de auto-refresco del bloque';
 $string['auto_refresh'] = 'Frecuencia de auto-refresco';
 $string['auto_refresh_description'] = 'Tiempo en milisegundos. Escriba "0" para deshabilitar el auto-refresco.';
 
-//Capabilities
+// Strings for capabilities.
 $string['ejsapp_file_browser:addinstance'] = 'Añadir un nuevo bloque de ficheros privados para EJSApp';
 $string['ejsapp_file_browser:myaddinstance'] = 'Añadir un nuevo bloque de ficheros privados para EJSApp al &Aacute;rea personal';