aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Marien Fressinaud <dev@marienfressinaud.fr> 2015-01-14 23:40:18 +0100
committerGravatar Marien Fressinaud <dev@marienfressinaud.fr> 2015-01-14 23:40:18 +0100
commit789d9fc6c8647fa100d80e3ef52727c813f35f16 (patch)
tree4367da6f708fbc66cb446a6dc8daf7b88a3f2293
parent2f1116818d7d9a6ef413b2c06e9603ffceaf7867 (diff)
Fix coding style in extension manager
Yep, same as 8968288...
-rw-r--r--app/Controllers/extensionController.php10
-rw-r--r--app/FreshRSS.php2
-rw-r--r--lib/Minz/ExtensionManager.php10
3 files changed, 11 insertions, 11 deletions
diff --git a/app/Controllers/extensionController.php b/app/Controllers/extensionController.php
index ec5eecadb..b6d2d3fe4 100644
--- a/app/Controllers/extensionController.php
+++ b/app/Controllers/extensionController.php
@@ -25,7 +25,7 @@ class FreshRSS_extension_Controller extends Minz_ActionController {
'user' => array(),
);
- $extensions = Minz_ExtensionManager::list_extensions();
+ $extensions = Minz_ExtensionManager::listExtensions();
foreach ($extensions as $ext) {
$this->view->extension_list[$ext->getType()][] = $ext;
}
@@ -50,7 +50,7 @@ class FreshRSS_extension_Controller extends Minz_ActionController {
}
$ext_name = urldecode(Minz_Request::param('e'));
- $ext = Minz_ExtensionManager::find_extension($ext_name);
+ $ext = Minz_ExtensionManager::findExtension($ext_name);
if (is_null($ext)) {
Minz_Error::error(404);
@@ -77,7 +77,7 @@ class FreshRSS_extension_Controller extends Minz_ActionController {
if (Minz_Request::isPost()) {
$ext_name = urldecode(Minz_Request::param('e'));
- $ext = Minz_ExtensionManager::find_extension($ext_name);
+ $ext = Minz_ExtensionManager::findExtension($ext_name);
if (is_null($ext)) {
Minz_Request::bad(_t('feedback.extensions.not_found', $ext_name),
@@ -133,7 +133,7 @@ class FreshRSS_extension_Controller extends Minz_ActionController {
if (Minz_Request::isPost()) {
$ext_name = urldecode(Minz_Request::param('e'));
- $ext = Minz_ExtensionManager::find_extension($ext_name);
+ $ext = Minz_ExtensionManager::findExtension($ext_name);
if (is_null($ext)) {
Minz_Request::bad(_t('feedback.extensions.not_found', $ext_name),
@@ -193,7 +193,7 @@ class FreshRSS_extension_Controller extends Minz_ActionController {
if (Minz_Request::isPost()) {
$ext_name = urldecode(Minz_Request::param('e'));
- $ext = Minz_ExtensionManager::find_extension($ext_name);
+ $ext = Minz_ExtensionManager::findExtension($ext_name);
if (is_null($ext)) {
Minz_Request::bad(_t('feedback.extensions.not_found', $ext_name),
diff --git a/app/FreshRSS.php b/app/FreshRSS.php
index 785b10299..021687999 100644
--- a/app/FreshRSS.php
+++ b/app/FreshRSS.php
@@ -53,7 +53,7 @@ class FreshRSS extends Minz_FrontController {
// Enable extensions for the current (logged) user.
if (FreshRSS_Auth::hasAccess()) {
$ext_list = FreshRSS_Context::$user_conf->extensions_enabled;
- Minz_ExtensionManager::enable_by_list($ext_list);
+ Minz_ExtensionManager::enableByList($ext_list);
}
}
diff --git a/lib/Minz/ExtensionManager.php b/lib/Minz/ExtensionManager.php
index 3df72abb7..8369e242a 100644
--- a/lib/Minz/ExtensionManager.php
+++ b/lib/Minz/ExtensionManager.php
@@ -52,7 +52,7 @@ class Minz_ExtensionManager {
}
$meta_raw_content = file_get_contents($metadata_filename);
$meta_json = json_decode($meta_raw_content, true);
- if (!$meta_json || !self::is_valid_metadata($meta_json)) {
+ if (!$meta_json || !self::isValidMetadata($meta_json)) {
// metadata.json is not a json file? Invalid!
// or metadata.json is invalid (no required information), invalid!
Minz_Log::warning('`' . $metadata_filename . '` is not a valid metadata file');
@@ -81,7 +81,7 @@ class Minz_ExtensionManager {
* @param $meta is an array of values.
* @return true if the array is valid, false else.
*/
- public static function is_valid_metadata($meta) {
+ public static function isValidMetadata($meta) {
$valid_chars = array('_');
return !(empty($meta['name']) ||
empty($meta['entrypoint']) ||
@@ -168,7 +168,7 @@ class Minz_ExtensionManager {
*
* @param $ext_list the names of extensions we want to load.
*/
- public static function enable_by_list($ext_list) {
+ public static function enableByList($ext_list) {
foreach ($ext_list as $ext_name) {
self::enable($ext_name);
}
@@ -180,7 +180,7 @@ class Minz_ExtensionManager {
* @param $only_enabled if true returns only the enabled extensions (false by default).
* @return an array of extensions.
*/
- public static function list_extensions($only_enabled = false) {
+ public static function listExtensions($only_enabled = false) {
if ($only_enabled) {
return self::$ext_list_enabled;
} else {
@@ -194,7 +194,7 @@ class Minz_ExtensionManager {
* @param $ext_name the name of the extension.
* @return the corresponding extension or null if it doesn't exist.
*/
- public static function find_extension($ext_name) {
+ public static function findExtension($ext_name) {
if (!isset(self::$ext_list[$ext_name])) {
return null;
}