YOURLS / YOURLS

admin/plugins.php

<?php
define( 'YOURLS_ADMIN', true );
require_once( dirname( __DIR__ ).'/includes/load-yourls.php' );
yourls_maybe_require_auth();

// Handle plugin administration pages
if( isset( $_GET['page'] ) && !empty( $_GET['page'] ) ) {
	yourls_plugin_admin_page( $_GET['page'] );
    die();
}

// Handle activation/deactivation of plugins
if( isset( $_GET['action'] ) ) {

	// Check nonce
	yourls_verify_nonce( 'manage_plugins', $_REQUEST['nonce'] );

	// Check plugin file is valid
	if( isset( $_GET['plugin'] ) && yourls_validate_plugin_file( YOURLS_PLUGINDIR.'/'.$_GET['plugin'].'/plugin.php') ) {

This line exceeds maximum limit of 100 characters; contains 117 characters

		// Activate / Deactive
		switch( $_GET['action'] ) {
			case 'activate':
				$result = yourls_activate_plugin( $_GET['plugin'].'/plugin.php' );
				if( $result === true )
					yourls_redirect( yourls_admin_url( 'plugins.php?success=activated' ), 302 );

				break;

Case breaking statement indented incorrectly; expected 7 spaces, found 4

			case 'deactivate':
				$result = yourls_deactivate_plugin( $_GET['plugin'].'/plugin.php' );
				if( $result === true )
					yourls_redirect( yourls_admin_url( 'plugins.php?success=deactivated' ), 302 );

				break;

Case breaking statement indented incorrectly; expected 7 spaces, found 4

			default:
				$result = yourls__( 'Unsupported action' );
				break;

Case breaking statement indented incorrectly; expected 7 spaces, found 4

		}
	} else {
		$result = yourls__( 'No plugin specified, or not a valid plugin' );
	}

	yourls_add_notice( $result );
}

// Handle message upon succesfull (de)activation
if( isset( $_GET['success'] ) && ( ( $_GET['success'] == 'activated' ) OR ( $_GET['success'] == 'deactivated' ) ) ) {

Operator == prohibited; use === instead

Using logical operators such as or instead of || is generally not recommended.

As per coding-style, PHP keywords should be in lowercase; expected or, but found OR.

This line exceeds maximum limit of 100 characters; contains 117 characters

	if( $_GET['success'] == 'activated' ) {

Operator == prohibited; use === instead

		$message = yourls__( 'Plugin has been activated' );
	} elseif ( $_GET['success'] == 'deactivated' ) {

Operator == prohibited; use === instead

		$message = yourls__( 'Plugin has been deactivated' );
	}
	yourls_add_notice( $message );
}

yourls_html_head( 'plugins', yourls__( 'Manage Plugins' ) );
yourls_html_logo();
yourls_html_menu();
?>

	<main role="main">
	<h2><?php yourls_e( 'Plugins' ); ?></h2>

	<?php
	$plugins = (array)yourls_get_plugins();
	uasort( $plugins, 'yourls_plugins_sort_callback' );

	$count = count( $plugins );
	$plugins_count = sprintf( yourls_n( '%s plugin', '%s plugins', $count ), $count );
	$count_active = yourls_has_active_plugins();
	?>

	<p id="plugin_summary"><?php /* //translators: "you have '3 plugins' installed and '1' activated" */ yourls_se( 'You currently have <strong>%1$s</strong> installed, and <strong>%2$s</strong> activated', $plugins_count, $count_active ); ?></p>

Single line block comment not allowed; use inline ("// text") comment instead

This line exceeds maximum limit of 100 characters; contains 243 characters

	<table id="main_table" class="tblSorter" cellpadding="0" cellspacing="1">
	<thead>
		<tr>
			<th><?php yourls_e( 'Plugin Name' ); ?></th>
			<th><?php yourls_e( 'Version' ); ?></th>
			<th><?php yourls_e( 'Description' ); ?></th>
			<th><?php yourls_e( 'Author' ); ?></th>
			<th><?php yourls_e( 'Action' ); ?></th>
		</tr>
	</thead>
	<tbody>
	<?php

	$nonce = yourls_create_nonce( 'manage_plugins' );

	foreach( $plugins as $file=>$plugin ) {

		// default fields to read from the plugin header
		$fields = array(
			'name'       => 'Plugin Name',
			'uri'        => 'Plugin URI',
			'desc'       => 'Description',
			'version'    => 'Version',
			'author'     => 'Author',
			'author_uri' => 'Author URI'

Each line in an array declaration must end in a comma

		);

The closing parenthesis does not seem to be aligned correctly; expected 12 space(s), but found 2.

		// Loop through all default fields, get value if any and reset it
		foreach( $fields as $field=>$value ) {
			if( isset( $plugin[ $value ] ) ) {
				$data[ $field ] = $plugin[ $value ];
			} else {
				$data[ $field ] = yourls__('(no info)');
			}
			unset( $plugin[$value] );
		}

		$plugindir = trim( dirname( $file ), '/' );

		if( yourls_is_active_plugin( $file ) ) {
			$class = 'active';
			$action_url = yourls_nonce_url( 'manage_plugins', yourls_add_query_arg( array('action' => 'deactivate', 'plugin' => $plugindir ), yourls_admin_url('plugins.php') ) );
			$action_anchor = yourls__( 'Deactivate' );
		} else {
			$class = 'inactive';
			$action_url = yourls_nonce_url( 'manage_plugins', yourls_add_query_arg( array('action' => 'activate', 'plugin' => $plugindir ), yourls_admin_url('plugins.php') ) );
			$action_anchor = yourls__( 'Activate' );
		}

		// Other "Fields: Value" in the header? Get them too
		if( $plugin ) {
			foreach( $plugin as $extra_field=>$extra_value ) {
				$data['desc'] .= "<br/>\n<em>$extra_field</em>: $extra_value";

As per coding-style, please use concatenation or sprintf for the variable $extra_field instead of interpolation.

As per coding-style, please use concatenation or sprintf for the variable $extra_value instead of interpolation.

				unset( $plugin[$extra_value] );
			}
		}

		$data['desc'] .= '<br/><small>' . yourls_s( 'plugin file location: %s', $file) . '</small>';

		printf( "<tr class='plugin %s'><td class='plugin_name'><a href='%s'>%s</a></td><td class='plugin_version'>%s</td><td class='plugin_desc'>%s</td><td class='plugin_author'><a href='%s'>%s</a></td><td class='plugin_actions actions'><a href='%s'>%s</a></td></tr>",

This line exceeds maximum limit of 100 characters; contains 262 characters

			$class, $data['uri'], $data['name'], $data['version'], $data['desc'], $data['author_uri'], $data['author'], $action_url, $action_anchor

This line exceeds maximum limit of 100 characters; contains 138 characters

			);

	}
	?>
	</tbody>
	</table>

	<script type="text/javascript">
	yourls_defaultsort = 0;
	yourls_defaultorder = 0;
	<?php if ($count_active) { ?>
	$('#plugin_summary').append('<span id="toggle_plugins">filter</span>');
	$('#toggle_plugins').css({'background':'transparent url("../images/filter.gif") top left no-repeat','display':'inline-block','text-indent':'-9999px','width':'16px','height':'16px','margin-left':'3px','cursor':'pointer'})

This line exceeds maximum limit of 100 characters; contains 221 characters

		.attr('title', '<?php echo yourls_esc_attr__( 'Toggle active/inactive plugins' ); ?>')
		.click(function(){
			$('#main_table tr.inactive').toggle();
		});
	<?php } ?>
	</script>

	<p><?php yourls_e( 'If something goes wrong after you activate a plugin and you cannot use YOURLS or access this page, simply rename or delete its directory, or rename the plugin file to something different than <code>plugin.php</code>.' ); ?></p>

This line exceeds maximum limit of 100 characters; contains 248 characters

	<h3><?php yourls_e( 'More plugins' ); ?></h3>

	<p><?php yourls_e( 'For more plugins, head to the official <a href="http://yourls.org/awesome">Plugin list</a>.' ); ?></p>
	</main>

<?php yourls_html_footer(); ?>