Rumperuu/footnotes
Archived
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

refactor: change OrIfs to early returns

Browse source
This commit is contained in:
Ben Goldsworthy 2021-05-02 20:00:55 +01:00
parent 39871b6cd0
commit c25bd7d8a2
3 changed files with 10 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/admin/layout/class-engine.php
View file

@ -572,7 +572,7 @@ abstract class Engine {
$p_in_min, $p_in_min,
$p_int_max $p_int_max
); );
} else { }
return sprintf( return sprintf(
'<input type="number" name="%s" id="%s" value="%d" min="%d" max="%d"/>', '<input type="number" name="%s" id="%s" value="%d" min="%d" max="%d"/>',
$l_arr_data['name'], $l_arr_data['name'],
@ -582,6 +582,5 @@ abstract class Engine {
$p_int_max $p_int_max
); );
} }
}
} }

5
src/includes/class-convert.php
View file

@ -185,7 +185,10 @@ class Convert {
$p_int_index = (int) $p_int_index; $p_int_index = (int) $p_int_index;
} }
// Return the whole arrow array. // Return the whole arrow array.
if ( $p_int_index < 0 || $p_int_index > count( $l_arr_arrows ) ) { if ($p_int_index < 0) {
return $l_arr_arrows;
}
if ($p_int_index > count( $l_arr_arrows )) {
return $l_arr_arrows; return $l_arr_arrows;
} }
// Return a single arrow. // Return a single arrow.

6
src/includes/class-template.php
View file

@ -81,10 +81,12 @@ class Template {
*/ */
public function __construct( string $p_str_file_type, string $p_str_file_name, string $p_str_extension = 'html' ) { public function __construct( string $p_str_file_type, string $p_str_file_name, string $p_str_extension = 'html' ) {
// No template file type and/or file name set. // No template file type and/or file name set.
if ( empty( $p_str_file_type ) || empty( $p_str_file_name ) ) { if (empty( $p_str_file_type )) {
return;
}
if (empty( $p_str_file_name )) {
return; return;
} }
$this->plugin_directory = plugin_dir_path( __DIR__ ); $this->plugin_directory = plugin_dir_path( __DIR__ );
$template = $this->get_template( $p_str_file_type, $p_str_file_name, $p_str_extension ); $template = $this->get_template( $p_str_file_type, $p_str_file_name, $p_str_extension );