GravityView_View

Complexity

Total Complexity

115

Size/Duplication

Total Lines	938
Duplicated Lines	0 %

Coupling/Cohesion

Components	5
Dependencies	6

Test Coverage

Coverage

88.11%

Importance

Changes

0

50 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	0	8	2
A	getCurrentFieldSetting()	0	9	3
A	getForm()	0	3	1
A	getPostId()	0	3	1
A	getViewId()	0	3	1
A	getFormId()	0	3	1
A	getEntries()	0	3	1
A	getTotalEntries()	0	3	1
A	getPaging()	0	9	1
B	getPaginationCounts()	0	27	4
B	__construct()	0	35	3
A	setCurrentFieldSetting()	0	7	2
A	getAtts()	0	11	3
A	setAtts()	0	3	1
A	setForm()	0	3	1
A	setPostId()	0	3	1
A	getContext()	0	3	1
A	setContext()	0	3	1
B	getFields()	0	10	5
B	getContextFields()	0	20	5
A	setFields()	0	3	1
A	getField()	0	10	3
A	setField()	0	3	1
A	setViewId()	0	3	1
A	setFormId()	0	3	1
A	setEntries()	0	3	1
A	setTotalEntries()	0	3	1
A	setPaging()	0	3	1
A	getSorting()	0	10	1
A	setSorting()	0	3	1
A	getBackLinkLabel()	0	8	2
A	setBackLinkLabel()	0	3	1
A	isHideUntilSearched()	0	3	1
A	setHideUntilSearched()	0	3	1
A	getTemplatePartSlug()	0	3	1
A	setTemplatePartSlug()	0	3	1
A	getTemplatePartName()	0	3	1
A	setTemplatePartName()	0	3	1
A	setCurrentEntry()	0	3	1
A	clearCurrentEntry()	0	3	1
C	renderZone()	0	76	10
B	locate_template()	0	22	6
A	__get()	0	7	2
B	add_id_specific_templates()	0	25	3
A	render()	0	20	3
F	render_widget_hooks()	0	108	19
A	_include()	0	5	2
A	getCurrentField()	0	11	3
A	setCurrentField()	0	14	1
A	getCurrentEntry()	0	16	4

<?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 23 and the first side effect is on line 16.

/**
 * GravityView templating engine class
 *
 * @package   GravityView
 * @license   GPL2+
 * @author    Katz Web Services, Inc.
 * @link      http://gravityview.co
 * @copyright Copyright 2014, Katz Web Services, Inc.
 *
 * @since 1.0.0
 */

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

if( ! class_exists( 'Gamajo_Template_Loader' ) ) {
	require( GRAVITYVIEW_DIR . 'includes/lib/class-gamajo-template-loader.php' );
}

class GravityView_View extends Gamajo_Template_Loader {

	/**
	 * Prefix for filter names.
	 * @var string
	 */
	protected $filter_prefix = 'gravityview';

	/**
	 * Directory name where custom templates for this plugin should be found in the theme.
	 * @var string
	 */
	protected $theme_template_directory = 'gravityview';

	/**
	 * Reference to the root directory path of this plugin.
	 * @var string
	 */
	protected $plugin_directory = GRAVITYVIEW_DIR;

	/**
	 * Store templates locations that have already been located
	 * @var array
	 */
	protected $located_templates = array();

	/**
	 * The name of the template, like "list", "table", or "datatables"
	 * @var string
	 */
	protected $template_part_slug = '';

	/**
	 * The name of the file part, like "body" or "single"
	 * @var string
	 */
	protected $template_part_name = '';

	/**
	 * @var int Gravity Forms form ID
	 */
	protected $form_id = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
	 * @var int View ID
	 * @todo: this needs to be public until extensions support 1.7+
	 */
	public $view_id = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
	 * @var array Fields for the form
	 */
	protected $fields = array();

	/**
	 * @var string Current screen. Defaults to "directory" or "single"
	 */
	protected $context = 'directory';

	/**
	 * @var int|null If in embedded post or page, the ID of it
	 */
	protected $post_id = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
	 * @var array Gravity Forms form array at ID $form_id
	 */
	protected $form = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
	 * @var array Configuration for the View
	 */
	protected $atts = array();

	/**
	 * @var array Entries for the current result. Single item in array for single entry View
	 */
	protected $entries = array();

	/**
	 * @var int Total entries count for the current result.
	 */
	protected $total_entries = 0;

	/**
	 * @var string The label to display back links
	 */
	protected $back_link_label = '';

	/**
	 * @var array Array with `offset` and `page_size` keys
	 */
	protected $paging = array();

	/**
	 * @var array Array with `sort_field` and `sort_direction` keys
	 */
	protected $sorting = array();

	/**
	 * @var bool Whether to hide the results until a search is performed
	 * @since 1.5.4
	 */
	protected $hide_until_searched = false;

	/**
	 * Current entry in the loop
	 * @var array
	 */
	protected $_current_entry = array();

	/**
	 * @var array
	 */
	protected $_current_field = array();

	/**
	 * @var GravityView_View
	 */
	static $instance = NULL;

The visibility should be declared for property $instance.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	/**
	 * Construct the view object
	 * @param  array       $atts Associative array to set the data of
	 */
	function __construct( $atts = array() ) {

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

		$atts = wp_parse_args( $atts, array(
			'form_id' => NULL,

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

			'view_id' => NULL,

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

			'fields'  => NULL,

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

			'context' => NULL,

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

			'post_id' => NULL,

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

			'form'    => NULL,

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

			'atts'	  => NULL,

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		) );

		foreach ($atts as $key => $value) {

No space after opening parenthesis is prohibited

No space before closing parenthesis is prohibited

			if( is_null( $value ) ) {
				continue;
			}
			$this->{$key} = $value;
		}

Functions must not contain multiple empty lines in a row; found 2 empty lines

		// Add granular overrides
		add_filter( $this->filter_prefix . '_get_template_part', array( $this, 'add_id_specific_templates' ), 10, 3 );

		// widget logic
		add_action( 'gravityview/template/before', array( $this, 'render_widget_hooks' ) );
		add_action( 'gravityview/template/after', array( $this, 'render_widget_hooks' ) );

		/**
		 * Clear the current entry after the loop is done
		 * @since 1.7.3
		 */
		add_action( 'gravityview_footer', array( $this, 'clearCurrentEntry' ), 500 );

		self::$instance = &$this;
	}

	/**
	 * @param null $passed_post
	 *
	 * @return GravityView_View
	 */
	static function getInstance( $passed_post = NULL ) {

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

The function name getInstance is in camel caps, but expected get_instance instead as per the coding standard.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		if( empty( self::$instance ) ) {
			self::$instance = new self( $passed_post );
		}

		return self::$instance;
	}

	/**
	 * @param string|null $key The key to a specific attribute of the current field
	 * @return array|mixed|null If $key is set and attribute exists at $key, return that. If not set, return NULL. Otherwise, return current field array
	 */
	public function getCurrentField( $key = NULL ) {

The function name getCurrentField is in camel caps, but expected get_current_field instead as per the coding standard.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		if( !empty( $key ) ) {

Expected 1 space after "!"; 0 found

			if( isset( $this->_current_field[ $key ] ) ) {
				return $this->_current_field[ $key ];
			}
			return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		}

		return $this->_current_field;
	}

	public function setCurrentFieldSetting( $key, $value ) {

The function name setCurrentFieldSetting is in camel caps, but expected set_current_field_setting instead as per the coding standard.

		if( !empty( $this->_current_field ) ) {

Expected 1 space after "!"; 0 found

			$this->_current_field['field_settings'][ $key ] = $value;
		}

	}

	public function getCurrentFieldSetting( $key ) {

The function name getCurrentFieldSetting is in camel caps, but expected get_current_field_setting instead as per the coding standard.

		$settings = $this->getCurrentField('field_settings');

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		if( $settings && !empty( $settings[ $key ] ) ) {

Expected 1 space after "!"; 0 found

			return $settings[ $key ];
		}

		return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	}

	/**
	 * @param array $passed_field
	 */
	public function setCurrentField( $passed_field ) {

The function name setCurrentField is in camel caps, but expected set_current_field instead as per the coding standard.

		$existing_field = $this->getCurrentField();

		$set_field = wp_parse_args( $passed_field, $existing_field );

		$this->_current_field = $set_field;

		/**
		 * Backward compatibility
		 * @deprecated 1.6.2
		 */
		$this->field_data = $set_field;

The property field_data does not exist. Did you maybe forget to declare it?

	}

	/**
	 * @param string|null $key The key to a specific field in the fields array
	 * @return array|mixed|null If $key is set and field exists at $key, return that. If not set, return NULL. Otherwise, return array of fields.
	 */
	public function getAtts( $key = NULL ) {

The function name getAtts is in camel caps, but expected get_atts instead as per the coding standard.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		if( !empty( $key ) ) {

Expected 1 space after "!"; 0 found

			if( isset( $this->atts[ $key ] ) ) {
				return $this->atts[ $key ];
			}
			return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		}

		return $this->atts;
	}

	/**
	 * @param array $atts
	 */
	public function setAtts( $atts ) {

The function name setAtts is in camel caps, but expected set_atts instead as per the coding standard.

		$this->atts = $atts;
	}

	/**
	 * @return array
	 */
	public function getForm() {

The function name getForm is in camel caps, but expected get_form instead as per the coding standard.

		return $this->form;
	}

	/**
	 * @param array $form
	 */
	public function setForm( $form ) {

The function name setForm is in camel caps, but expected set_form instead as per the coding standard.

		$this->form = $form;
	}

	/**
	 * @return int|null
	 */
	public function getPostId() {

The function name getPostId is in camel caps, but expected get_post_id instead as per the coding standard.

		return $this->post_id;
	}

	/**
	 * @param int|null $post_id
	 */
	public function setPostId( $post_id ) {

The function name setPostId is in camel caps, but expected set_post_id instead as per the coding standard.

		$this->post_id = $post_id;
	}

	/**
	 * @return string
	 */
	public function getContext() {

The function name getContext is in camel caps, but expected get_context instead as per the coding standard.

		return $this->context;
	}

	/**
	 * @param string $context
	 */
	public function setContext( $context ) {

The function name setContext is in camel caps, but expected set_context instead as per the coding standard.

		$this->context = $context;
	}

	/**
	 * @param string|null $key The key to a specific field in the fields array
	 * @return array|mixed|null If $key is set and field exists at $key, return that. If not set, return NULL. Otherwise, return array of fields.
	 */
	public function getFields( $key = null ) {

The function name getFields is in camel caps, but expected get_fields instead as per the coding standard.

		$fields = empty( $this->fields ) ? NULL : $this->fields;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		if( $fields && !empty( $key ) ) {

The expression $fields of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

Expected 1 space after "!"; 0 found

			$fields = isset( $fields[ $key ] ) ? $fields[ $key ] : NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		}

		return $fields;
	}

	/**
     * Get the fields for a specific context
     *
     * @since 1.19.2
     *
	 * @param string $context [Optional] "directory", "single", or "edit"
	 *
	 * @return array Array of GravityView field layout configurations
	 */
	public function getContextFields( $context = '' ) {

The function name getContextFields is in camel caps, but expected get_context_fields instead as per the coding standard.

	    if( '' === $context ) {
	        $context = $this->getContext();
        }

		$fields = $this->getFields();

        foreach ( (array) $fields as $key => $context_fields ) {

            // Formatted as `{context}_{template id}-{zone name}`, so we want just the $context to match against
            $matches = explode( '_', $key );

            if( isset( $matches[0] ) && $matches[0] === $context ) {
                return $context_fields;
            }
        }

		return array();
    }

	/**
	 * @param array $fields
	 */
	public function setFields( $fields ) {

The function name setFields is in camel caps, but expected set_fields instead as per the coding standard.

		$this->fields = $fields;
	}

	/**
	 * @param string $key The key to a specific field in the fields array
	 * @return array|mixed|null If $key is set and field exists at $key, return that. If not set, return NULL. Otherwise, return array of fields.
	 */
	public function getField( $key ) {

The function name getField is in camel caps, but expected get_field instead as per the coding standard.

		if( !empty( $key ) ) {

Expected 1 space after "!"; 0 found

			if( isset( $this->fields[ $key ] ) ) {
				return $this->fields[ $key ];
			}
		}

		return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	}

	/**
	 * @param string $key The key to a specific field in the fields array
	 * @param mixed $value The value to set for the field
	 */
	public function setField( $key, $value ) {

The function name setField is in camel caps, but expected set_field instead as per the coding standard.

		$this->fields[ $key ] = $value;
	}

	/**
	 * @return int
	 */
	public function getViewId() {

The function name getViewId is in camel caps, but expected get_view_id instead as per the coding standard.

		return absint( $this->view_id );
	}

	/**
	 * @param int $view_id
	 */
	public function setViewId( $view_id ) {

The function name setViewId is in camel caps, but expected set_view_id instead as per the coding standard.

		$this->view_id = intval( $view_id );
	}

	/**
	 * @return int
	 */
	public function getFormId() {

The function name getFormId is in camel caps, but expected get_form_id instead as per the coding standard.

		return $this->form_id;
	}

	/**
	 * @param int $form_id
	 */
	public function setFormId( $form_id ) {

The function name setFormId is in camel caps, but expected set_form_id instead as per the coding standard.

		$this->form_id = $form_id;
	}

	/**
	 * @return array
	 */
	public function getEntries() {

The function name getEntries is in camel caps, but expected get_entries instead as per the coding standard.

		return $this->entries;
	}

	/**
	 * @param array $entries
	 */
	public function setEntries( $entries ) {

The function name setEntries is in camel caps, but expected set_entries instead as per the coding standard.

		$this->entries = $entries;
	}

	/**
	 * @return int
	 */
	public function getTotalEntries() {

The function name getTotalEntries is in camel caps, but expected get_total_entries instead as per the coding standard.

		return (int)$this->total_entries;

No space after closing casting parenthesis is prohibited

	}

	/**
	 * @param int $total_entries
	 */
	public function setTotalEntries( $total_entries ) {

The function name setTotalEntries is in camel caps, but expected set_total_entries instead as per the coding standard.

		$this->total_entries = intval( $total_entries );
	}

	/**
	 * @return array
	 */
	public function getPaging() {

The function name getPaging is in camel caps, but expected get_paging instead as per the coding standard.

	    $default_params = array(
            'offset' => 0,
            'page_size' => 20,
        );

		return wp_parse_args( $this->paging, $default_params );
	}

	/**
	 * @param array $paging
	 */
	public function setPaging( $paging ) {

The function name setPaging is in camel caps, but expected set_paging instead as per the coding standard.

		$this->paging = $paging;
	}

	/**
	 * Get an array with pagination information
	 *
	 * @since 1.13
	 * 
	 * @return array {
	 *  @type int $first The starting entry number (counter, not ID)
	 *  @type int $last The last displayed entry number (counter, not ID)
	 *  @type int $total The total number of entries
	 * }
	 */
	public function getPaginationCounts() {

The function name getPaginationCounts is in camel caps, but expected get_pagination_counts instead as per the coding standard.

		$paging = $this->getPaging();
		$offset = $paging['offset'];
		$page_size = $paging['page_size'];
		$total = $this->getTotalEntries();

		if ( empty( $total ) ) {
			gravityview()->log->debug( 'No entries. Returning empty array.' );

			return array();
		}

		$first = empty( $offset ) ? 1 : $offset + 1;

		// If the page size + starting entry is larger than total, the total is the max.
		$last = ( $offset + $page_size > $total ) ? $total : $offset + $page_size;

		/**
		 * @filter `gravityview_pagination_counts` Modify the displayed pagination numbers
		 * @since 1.13
		 * @param array $counts Array with $first, $last, $total numbers in that order
		 */
		list( $first, $last, $total ) = apply_filters( 'gravityview_pagination_counts', array( $first, $last, $total ) );

		return array( 'first' => (int) $first, 'last' => (int) $last, 'total' => (int) $total );
	}

	/**
	 * @return array
	 */
	public function getSorting() {

The function name getSorting is in camel caps, but expected get_sorting instead as per the coding standard.

		$defaults_params = array(
            'sort_field' => 'date_created',
            'sort_direction' => 'ASC',
            'is_numeric' => false,
        );

		return wp_parse_args( $this->sorting, $defaults_params );
	}

	/**
	 * @param array $sorting
	 */
	public function setSorting( $sorting ) {

The function name setSorting is in camel caps, but expected set_sorting instead as per the coding standard.

		$this->sorting = $sorting;
	}

	/**
	 * @param boolean $do_replace Perform merge tag and shortcode processing on the label. Default: true.
	 * @since 2.0
	 *
	 * @deprecated Use $template->get_back_label();
	 *
	 * @return string
	 */
	public function getBackLinkLabel( $do_replace = true ) {

The function name getBackLinkLabel is in camel caps, but expected get_back_link_label instead as per the coding standard.

		if ( $do_replace ) {
			$back_link_label = GravityView_API::replace_variables( $this->back_link_label, $this->getForm(), $this->getCurrentEntry() );
			return do_shortcode( $back_link_label );
		}

		return $this->back_link_label;
	}

	/**
	 * @param string $back_link_label
	 */
	public function setBackLinkLabel( $back_link_label ) {

The function name setBackLinkLabel is in camel caps, but expected set_back_link_label instead as per the coding standard.

		$this->back_link_label = $back_link_label;
	}

	/**
	 * @return boolean
	 */
	public function isHideUntilSearched() {

The function name isHideUntilSearched is in camel caps, but expected is_hide_until_searched instead as per the coding standard.

		return $this->hide_until_searched;
	}

	/**
	 * @param boolean $hide_until_searched
	 */
	public function setHideUntilSearched( $hide_until_searched ) {

The function name setHideUntilSearched is in camel caps, but expected set_hide_until_searched instead as per the coding standard.

		$this->hide_until_searched = $hide_until_searched;
	}

	/**
	 * @return string
	 */
	public function getTemplatePartSlug() {

The function name getTemplatePartSlug is in camel caps, but expected get_template_part_slug instead as per the coding standard.

		return $this->template_part_slug;
	}

	/**
	 * @param string $template_part_slug
	 */
	public function setTemplatePartSlug( $template_part_slug ) {

The function name setTemplatePartSlug is in camel caps, but expected set_template_part_slug instead as per the coding standard.

		$this->template_part_slug = $template_part_slug;
	}

	/**
	 * @return string
	 */
	public function getTemplatePartName() {

The function name getTemplatePartName is in camel caps, but expected get_template_part_name instead as per the coding standard.

		return $this->template_part_name;
	}

	/**
	 * @param string $template_part_name
	 */
	public function setTemplatePartName( $template_part_name ) {

The function name setTemplatePartName is in camel caps, but expected set_template_part_name instead as per the coding standard.

		$this->template_part_name = $template_part_name;
	}

	/**
	 * Return the current entry. If in the loop, the current entry. If single entry, the currently viewed entry.
	 * @return array
	 */
	public function getCurrentEntry() {

The function name getCurrentEntry is in camel caps, but expected get_current_entry instead as per the coding standard.

		if( in_array( $this->getContext(), array( 'edit', 'single') ) ) {

No space before closing parenthesis of array is bad style

			$entries = $this->getEntries();
			$entry = $entries[0];
		} else {
			$entry = $this->_current_entry;
		}

		/** @since 1.16 Fixes DataTables empty entry issue */
		if ( empty( $entry ) && ! empty( $this->_current_field['entry'] ) ) {
			$entry = $this->_current_field['entry'];
		}

		return $entry;
	}

	/**
	 * @param array $current_entry
	 * @return void
	 */
	public function setCurrentEntry( $current_entry ) {

The function name setCurrentEntry is in camel caps, but expected set_current_entry instead as per the coding standard.

		$this->_current_entry = $current_entry;
	}

	/**
	 * Clear the current entry after all entries in the loop have been displayed.
	 *
	 * @since 1.7.3
	 * @return void
	 */
	public function clearCurrentEntry() {

The function name clearCurrentEntry is in camel caps, but expected clear_current_entry instead as per the coding standard.

		$this->_current_entry = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	}

	/**
	 * Render an output zone, as configured in the Admin
	 *
	 * @since 1.16.4 Added $echo parameter
	 *
	 * @param string $zone The zone name, like 'footer-left'
	 * @param array $atts
	 * @param bool $echo Whether to print the output
	 *
	 * @deprecated This will never get called in new templates.
	 *
	 * @return string|null
	 */
	public function renderZone( $zone = '', $atts = array(), $echo = true ) {

The function name renderZone is in camel caps, but expected render_zone instead as per the coding standard.

		if ( empty( $zone ) ) {
			gravityview()->log->error( 'No zone defined.');

Expected 1 spaces before closing bracket; 0 found

			return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		}

		$defaults = array(
			'slug' => $this->getTemplatePartSlug(),
			'context' => $this->getContext(),
			'entry' => $this->getCurrentEntry(),
			'form' => $this->getForm(),
			'hide_empty' => $this->getAtts('hide_empty'),

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		);

		$final_atts = wp_parse_args( $atts, $defaults );

		$output = '';

		$final_atts['zone_id'] = "{$final_atts['context']}_{$final_atts['slug']}-{$zone}";

		$fields = $this->getField( $final_atts['zone_id'] );

		// Backward compatibility
		if ( 'table' === $this->getTemplatePartSlug() ) {
			/**
			 * @filter `gravityview_table_cells` Modify the fields displayed in a table
			 * @param array $fields
			 * @param \GravityView_View $this
			 * @deprecated Use `gravityview/template/table/fields`
			 */
			$fields = apply_filters("gravityview_table_cells", $fields, $this );

Expected 1 spaces after opening bracket; 0 found

The string literal gravityview_table_cells does not require double quotes, as per coding-style, please use single quotes.

		}

		if ( empty( $fields ) ) {

			gravityview()->log->error( 'Empty View configuration for this context.', array( 'data' => $fields ) );

			return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		}

		$field_output = '';
		foreach ( $fields as $field ) {
			$final_atts['field'] = $field;

			$field_output .= gravityview_field_output( $final_atts );
		}

		/**
		 * If a zone has no field output, choose whether to show wrapper
		 * False by default to keep backward compatibility
		 * @since 1.7.6
		 * @param boolean $hide_empty_zone Default: false
		 * @since 2.0
		 * @param \GV\Template_Context $context The context. Null here. Since this path is deprecated.
		 */
		if ( empty( $field_output ) && apply_filters( 'gravityview/render/hide-empty-zone', false, null ) ) {
			return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		}

		if( !empty( $final_atts['wrapper_class'] ) ) {

Expected 1 space after "!"; 0 found

			$output .= '<div class="'.gravityview_sanitize_html_class( $final_atts['wrapper_class'] ).'">';
		}

		$output .= $field_output;

		if( !empty( $final_atts['wrapper_class'] ) ) {

Expected 1 space after "!"; 0 found

			$output .= '</div>';
		}

		if( $echo ) {
			echo $output;

Expected next thing to be a escaping function, not '$output'

		}

		return $output;
	}

	/**
	 * In order to improve lookup times, we store located templates in a local array.
	 *
	 * This improves performance by up to 1/2 second on a 250 entry View with 7 columns showing
	 *
	 * @inheritdoc
	 * @see Gamajo_Template_Loader::locate_template()
	 * @return null|string NULL: Template not found; String: path to template
	 */
	function locate_template( $template_names, $load = false, $require_once = true ) {

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

		if( is_string( $template_names ) && isset( $this->located_templates[ $template_names ] ) ) {

			$located = $this->located_templates[ $template_names ];

		} else {

			// Set $load to always false so we handle it here.
			$located = parent::locate_template( $template_names, false, $require_once );

			if( is_string( $template_names ) ) {
				$this->located_templates[ $template_names ] = $located;
			}
		}

		if ( $load && $located ) {
			load_template( $located, $require_once );
		}

		return $located;
	}

	/**
	 * Magic Method: Instead of throwing an error when a variable isn't set, return null.
	 * @param  string      $name Key for the data retrieval.
	 * @return mixed|null    The stored data.
	 */
	public function __get( $name ) {
		if( isset( $this->{$name} ) ) {
			return $this->{$name};
		} else {
			return NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		}
	}

	/**
	 * Enable overrides of GravityView templates on a granular basis
	 *
	 * The loading order is:
	 *
	 * - view-[View ID]-table-footer.php
	 * - form-[Form ID]-table-footer.php
	 * - page-[ID of post or page where view is embedded]-table-footer.php
	 * - table-footer.php
	 *
	 * @see  Gamajo_Template_Loader::get_template_file_names() Where the filter is
	 * @param array $templates Existing list of templates.
	 * @param string $slug      Name of the template base, example: `table`, `list`, `datatables`, `map`
	 * @param string $name      Name of the template part, example: `body`, `footer`, `head`, `single`
	 *
	 * @return array $templates Modified template array, merged with existing $templates values
	 */
	function add_id_specific_templates( $templates, $slug, $name ) {

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

		$additional = array();

		// form-19-table-body.php
		$additional[] = sprintf( 'form-%d-%s-%s.php', $this->getFormId(), $slug, $name );

		if( $view_id = $this->getViewId() ) {
			// view-3-table-body.php
			$additional[] = sprintf( 'view-%d-%s-%s.php', $view_id, $slug, $name );
		}

		if( $this->getPostId() ) {

The expression $this->getPostId() of type integer|null is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

			// page-19-table-body.php
			$additional[] = sprintf( 'page-%d-%s-%s.php', $this->getPostId(), $slug, $name );
		}

		// Combine with existing table-body.php and table.php
		$templates = array_merge( $additional, $templates );

		gravityview()->log->debug( 'List of Template Files', array( 'data' => $templates ) );

		return $templates;
	}

	// Load the template
	public function render( $slug, $name, $require_once = true ) {

		$this->setTemplatePartSlug( $slug );

		$this->setTemplatePartName( $name );
		
		$template_file = $this->get_template_part( $slug, $name, false );

		gravityview()->log->debug( 'Rendering Template File: {path}', array( 'path' => $template_file ) );

		if( !empty( $template_file) ) {

Expected 1 space after "!"; 0 found

Expected 1 spaces before closing bracket; 0 found

			if ( $require_once ) {
				require_once( $template_file );
			} else {
				require( $template_file );
			}

		}
	}

	/**
	 * Output the widgets on before/after hooks.
	 *
	 * @param int|\GV\Template_Context $view_id_or_context The View ID or the context.
	 *
	 * @return void
	 */
	public function render_widget_hooks( $view_id_or_context ) {

	    /**
		 * @deprecated Numeric argument is deprecated. Pass a \GV\Template_Context instead.
		 */
		if ( is_numeric( $view_id_or_context ) ) {
			$view = \GV\View::by_id( $view_id_or_context );
			$is_single = gravityview_get_context() == 'single';
			$total_entries = GravityView_View::getInstance()->getTotalEntries();

		} else if ( $view_id_or_context instanceof \GV\Template_Context ) {
			$view = $view_id_or_context->view;
			$is_single = (boolean)$view_id_or_context->request->is_entry();

No space after closing casting parenthesis is prohibited

			$total_entries = $view_id_or_context->entries ? $view_id_or_context->entries->count() : 0;

		} else {
			gravityview()->log->error( 'No View ID or template context provided to render_widget_hooks' );
			return;
		}

		if ( $is_single ) {
			gravityview()->log->debug( 'Not rendering widgets; single entry' );
			return;
		}

		switch ( current_filter() ) {
			default:
			case 'gravityview/template/before':
			case 'gravityview_before':
				$zone = 'header';
				break;
			case 'gravityview/template/after':
			case 'gravityview_after':
				$zone = 'footer';
				break;
		}

		$widgets = $view->widgets->by_position( "$zone*" );

		/**
		 * Prevent output if no widgets to show.
		 * @since 1.16
		 */
		if ( ! $widgets->count() ) {
			gravityview()->log->debug( 'No widgets for View #{view_id} in zone {zone}', array( 'view_id' => $view->ID, 'zone' => $zone ) );
			return;
		}

		// Prevent being called twice
		if ( did_action( "gravityview/widgets/$zone/{$view->ID}/rendered" ) ) {
			gravityview()->log->debug( 'Not rendering {zone}; already rendered', array( 'zone' => $zone.'_'.$view->ID.'_widgets' ) );
			return;
		}

		$rows = \GV\Widget::get_default_widget_areas();

		// TODO: Move to sep. method, use an action instead
		wp_enqueue_style( 'gravityview_default_style' );

		$default_css_class = 'gv-grid gv-widgets-' . $zone;

		if ( ! $total_entries ) {
			$default_css_class .= ' gv-widgets-no-results';
		}

		/**
		 * @filter `gravityview/widgets/wrapper_css_class` The CSS class applied to the widget container `<div>`.
		 * @since 1.16.2
		 * @param string $css_class Default: `gv-grid gv-widgets-{zone}` where `{zone}` is replaced by the current `$zone` value. If the View has no results, adds ` gv-widgets-no-results`
		 * @param string $zone Current widget zone, either `header` or `footer`
		 * @param array $widgets Array of widget configurations for the current zone, as set by `gravityview_get_current_view_data()['widgets']`
		 */
		$css_class = apply_filters('gravityview/widgets/wrapper_css_class', $default_css_class, $zone, $widgets->as_configuration() );

Expected 1 spaces after opening bracket; 0 found

		$css_class = gravityview_sanitize_html_class( $css_class );

		// TODO Convert to partials
		?>
		<div class="<?php echo $css_class; ?>">

Expected next thing to be a escaping function, not '$css_class'

			<?php
			foreach( $rows as $row ) {
				foreach( $row as $col => $areas ) {
					$column = ( $col == '2-2' ) ? '1-2 gv-right' : "$col gv-left";
				?>
					<div class="gv-grid-col-<?php echo esc_attr( $column ); ?>">
						<?php
						if ( ! empty( $areas ) ) {
							foreach ( $areas as $area ) {
								foreach ( $widgets->by_position( $zone . '_' . $area['areaid'] )->all() as $widget ) {
									do_action( sprintf( 'gravityview/widgets/%s/render', $widget->get_widget_id() ), $widget->configuration->all(), null, $view_id_or_context );
								}
							}
						} ?>
					</div>
				<?php } // $row ?>
			<?php } // $rows ?>
		</div>

		<?php

		/**
		 * Prevent widgets from being called twice.
		 * Checking for loop_start prevents themes and plugins that pre-process shortcodes from triggering the action before displaying. Like, ahem, the Divi theme and WordPress SEO plugin
		 */
		if ( did_action( 'wp_head' ) ) {
			do_action( "gravityview/widgets/$zone/{$view->ID}/rendered" );
		}
	}

	/**
	 * Include a file inside this context.
	 *
	 * @param string $path A path to the legacy template to include.
	 *
	 * @return void
	 */
	public function _include( $path ) {
		if ( file_exists( $path ) ) {
			include $path;
		}
	}

}