GV\Form_Collection

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	add()	0	6	2
A	get()	0	8	3

<?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 6.

namespace GV;

/** If this file is called directly, abort. */
if ( ! defined( 'GRAVITYVIEW_DIR' ) ) {
	die();
}

/**
 * A collection of \GV\Form objects.
 */
class Form_Collection extends Collection {
	/**
	 * Add a \GV\Form to this collection.
	 *
	 * @param \GV\Form $form The form to add to the internal array.
	 *
	 * @throws \InvalidArgumentException if $form is not of type \GV\Form.
	 *
	 * @api
	 * @since future
	 * @return void
	 */
	public function add( $form ) {
		if ( ! $form instanceof Form ) {
			throw new \InvalidArgumentException( 'Form_Collections can only contain objects of type \GV\Form.' );
		}
		parent::add( $form );
	}

	/**
	 * Get a \GV\Form from this list.
	 *
	 * @param int $form_id The ID of the form to get.
	 * @param string $backend The form backend identifier, allows for multiple form backends in the future. Unused until then.
	 *
	 * @api
	 * @since future
	 *
	 * @return \GV\Form|null The \GV\Form with the $form_id as the ID, or null if not found.
	 */
	public function get( $form_id, $backend = 'gravityforms' ) {

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

		foreach ( $this->all() as $form ) {
			if ( $form->ID == $form_id ) {
				return $form;
			}
		}
		return null;
	}
}