Give_Form_API

Complexity

Total Complexity

8

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	2	1
A	make_tag()	0	2	1
A	get_elements()	0	2	1
A	is_button()	0	3	1
A	render_form()	0	2	1
A	render_element()	0	2	1
A	process_form()	0	2	1
A	process_element()	0	2	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 12 and the first side effect is on line 112.

/**
 * Form API
 *
 * @package     Give
 * @subpackage  Classes/Give_Form_API
 * @copyright   Copyright (c) 2016, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       1.9
 */
class Give_Form_API {

	/**
	 * The defaults for all elements
	 *
	 * @since  1.9
	 * @access static
	 */
	static $field_defaults = array(

The visibility should be declared for property $field_defaults.

		'name'       => '',
		'desc'       => '',
		'id'         => '',
		'type'       => '',
		'default'    => '',
		'data_type'  => '',
		'options'    => array(),
		'attributes' => array(),
	);

	/**
	 * Initialize this module
	 *
	 * @since  1.9
	 * @access static
	 */
	static function init() {

It is generally recommended to explicitly declare the visibility for methods.

	}


	/**
	 * Return HTML with tag $tagname and keyed attrs $attrs.
	 *
	 * @since  1.9
	 * @access static
	 */
	static function make_tag() {

It is generally recommended to explicitly declare the visibility for methods.

	}

	/**
	 * Get elements from a form.
	 *
	 * @since  1.9
	 * @access static
	 *
	 * @param array $form
	 */
	static function get_elements( $form ) {

The parameter $form is not used and could be removed.

It is generally recommended to explicitly declare the visibility for methods.

	}

	/**
	 * Is the element a button?
	 *
	 * @since  1.9
	 * @access static
	 *
	 * @param array $element
	 *
	 * @return bool

Should the return type not be integer?

	 */
	static function is_button( $element ) {

It is generally recommended to explicitly declare the visibility for methods.

		return preg_match( '/^button|submit$/', $element['#type'] );
	}

	/**
	 * Render forms.
	 *
	 * @since  1.9
	 * @access static
	 */
	static function render_form() {

It is generally recommended to explicitly declare the visibility for methods.

	}

	/**
	 * Render an element
	 * @since  1.9
	 * @access static
	 */
	static function render_element() {

It is generally recommended to explicitly declare the visibility for methods.

	}

	/**
	 * Process a form, filling in $values with what's been posted
	 *
	 * @since  1.9
	 * @access static
	 */
	static function process_form() {

It is generally recommended to explicitly declare the visibility for methods.

	}

	/**
	 * Recursively process a meta form element, filling in $values accordingly
	 *
	 * @since  1.9
	 * @access static
	 */
	static function process_element() {

It is generally recommended to explicitly declare the visibility for methods.

	}
}

// Initialize field API.
add_action( 'init', array( 'Give_Form_API', 'init' ) );