podlove / podlove-subscribe-button-wp-plugin

version.php

<?php 
/**
 * Version management for database migrations.
 * 
 * Database changes require special care:
 * - the model has to be adjusted for users installing the plugin
 * - the current setup has to be migrated for current users
 * 
 * These migrations are a way to handle current users. They do *not*
 * run on plugin activation.
 * 
 * Pattern:
 * 
 * - increment \PodloveSubscribeButton\DATABASE_VERSION constant by 1, e.g.
 * 		```php

Spaces must be used for mid-line alignment; tabs are not allowed

 * 		define( __NAMESPACE__ . '\DATABASE_VERSION', 2 );

Spaces must be used for mid-line alignment; tabs are not allowed

 * 		```

Spaces must be used for mid-line alignment; tabs are not allowed

 * 		

Spaces must be used for mid-line alignment; tabs are not allowed

 * - add a case in `\PodloveSubscribeButton\run_migrations_for_version`, e.g.
 * 		```php

Spaces must be used for mid-line alignment; tabs are not allowed

 * 		function run_migrations_for_version( $version ) {

Spaces must be used for mid-line alignment; tabs are not allowed

 *			global $wpdb;

Spaces must be used for mid-line alignment; tabs are not allowed

 *			switch ( $version ) {

Spaces must be used for mid-line alignment; tabs are not allowed

 *				case 2:

Spaces must be used for mid-line alignment; tabs are not allowed

 *					$wbdb-> // run sql or whatever

Spaces must be used for mid-line alignment; tabs are not allowed

 *					break;

Spaces must be used for mid-line alignment; tabs are not allowed

 *			}

Spaces must be used for mid-line alignment; tabs are not allowed

 *		}

Spaces must be used for mid-line alignment; tabs are not allowed

 *		```

Spaces must be used for mid-line alignment; tabs are not allowed

 *		

Spaces must be used for mid-line alignment; tabs are not allowed

 *		Feel free to move the migration code into a separate function if it's

Spaces must be used for mid-line alignment; tabs are not allowed

 *		rather complex.

Spaces must be used for mid-line alignment; tabs are not allowed

 *		

Spaces must be used for mid-line alignment; tabs are not allowed

 * - adjust the main model / setup process so new users installing the plugin
 *   will have these changes too
 *   
 * - Test the migrations! :)
 */

Missing @package tag in file comment

namespace PodloveSubscribeButton;
use \PodloveSubscribeButton\Model;

The type \PodloveSubscribeButton\Model was not found. Maybe you did not declare it correctly or list all dependencies?

define( __NAMESPACE__ . '\DATABASE_VERSION', 2 );

add_action( 'admin_init', '\PodloveSubscribeButton\maybe_run_database_migrations' );
add_action( 'admin_init', '\PodloveSubscribeButton\run_database_migrations', 5 );

function maybe_run_database_migrations() {

Missing doc comment for function maybe_run_database_migrations()

	$database_version = get_option('podlove_subscribe_button_plugin_database_version');

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

	if ( $database_version === false ) {

Use Yoda Condition checks, you must.

		// plugin has just been installed or Plugin Version < 1.3

Inline comments must end in full-stops, exclamation marks, or question marks

		update_option( 'podlove_subscribe_button_plugin_database_version', DATABASE_VERSION );
	}
}

Expected 1 blank line before closing function brace; 0 found

function run_database_migrations() {

Missing doc comment for function run_database_migrations()

	if (get_option('podlove_subscribe_button_plugin_database_version') >= DATABASE_VERSION)

It is generally a best practice to always use braces with control structures.

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

		return;

	if (is_multisite()) {

No space after opening parenthesis is prohibited

No space before closing parenthesis is prohibited

		set_time_limit(0); // may take a while, depending on network size

Inline comments must end in full-stops, exclamation marks, or question marks

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

		\PodloveSubscribeButton\for_every_podcast_blog(function() { migrate_for_current_blog(); });

Opening brace must be the last content on the line

Closing brace of nested function must be on a new line

It is generally recommended to place each PHP statement on a line by itself.

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

	} else {
		migrate_for_current_blog();
	}

	if (isset($_REQUEST['_wp_http_referer']) && $_REQUEST['_wp_http_referer']) {

No space after opening parenthesis is prohibited

Processing form data without nonce verification.

Missing wp_unslash() before sanitization.

Detected usage of a non-sanitized input variable: $_REQUEST

No space before closing parenthesis is prohibited

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

		wp_redirect($_REQUEST['_wp_http_referer']);

wp_redirect() found. Using wp_safe_redirect(), along with the allowed_redirect_hosts filter if needed, can help avoid any chances of malicious redirects within code. It is also important to remember to call exit() after a redirect so that no other unwanted code is executed.

Missing wp_unslash() before sanitization.

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

		exit;
	}
}

Expected 1 blank line before closing function brace; 0 found

function migrate_for_current_blog() {

Missing doc comment for function migrate_for_current_blog()

	$database_version = get_option('podlove_subscribe_button_plugin_database_version');

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

	for ($i = $database_version+1; $i <= DATABASE_VERSION; $i++) { 

No space after opening parenthesis is prohibited

Expected 1 space before "+"; 0 found

Expected 1 space after "+"; 0 found

No space before closing parenthesis is prohibited

		\PodloveSubscribeButton\run_migrations_for_version($i);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

		update_option('podlove_subscribe_button_plugin_database_version', $i);

Expected 1 spaces after opening parenthesis; 0 found

Expected 1 spaces before closing parenthesis; 0 found

	}
}

Expected 1 blank line before closing function brace; 0 found

/**
 * Find and run migration for given version number.
 *
 * @todo  move migrations into separate files
 * 
 * @param  int $version

Missing parameter comment

 */
function run_migrations_for_version( $version ) {
	global $wpdb;
	
	switch ( $version ) {}

This switch statement is empty, and could be removed.

}