Completed
Push — develop ( bc21cb...e1c3d9 )
by Gennady
17:56
created

gravityview::detail()   A

Complexity

Conditions 5
Paths 5

Size

Total Lines 32

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 2
CRAP Score 16.7636

Importance

Changes 0
Metric Value
cc 5
nc 5
nop 3
dl 0
loc 32
ccs 2
cts 9
cp 0.2222
crap 16.7636
rs 9.0968
c 0
b 0
f 0
1
<?php
0 ignored issues
show
Coding Style Compatibility introduced by
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.

The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file.

The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place.

To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1.

Loading history...
2
namespace GV\Shortcodes;
3
4
/** If this file is called directly, abort. */
5
if ( ! defined( 'GRAVITYVIEW_DIR' ) ) {
6
	die();
7
}
8
9
/**
10
 * The [gravityview] shortcode.
11
 */
12
class gravityview extends \GV\Shortcode {
0 ignored issues
show
Coding Style introduced by
Class name "gravityview" is not in camel caps format
Loading history...
13
	/**
14
	 * {@inheritDoc}
15
	 */
16
	public $name = 'gravityview';
17
18
	/**
19
	 * Process and output the [gravityview] shortcode.
20
	 *
21
	 * @param array $passed_atts The attributes passed.
22
	 * @param string $content The content inside the shortcode.
23
	 *
24
	 * @return string|null The output.
25
	 */
26 6
	public function callback( $passed_atts, $content = null ) {
27
28 6
		$request = gravityview()->request;
29
30 6
		if ( $request->is_admin() ) {
31
			return '';
32
		}
33
34 6
		$atts = wp_parse_args( $passed_atts, array(
35 6
			'id' => 0,
36
			'view_id' => 0,
37
			'detail' => null,
38
		) );
39
		
40 6
		if ( ! $view_id = $atts['id'] ? : $atts['view_id'] ) {
41
			if ( $atts['detail'] && $view = $request->is_view() ) {
42
				$view_id = $view->ID;
0 ignored issues
show
Bug introduced by
The property ID does not seem to exist in GV\View.

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
43
			}
44
		}
45
46 6
		$view = \GV\View::by_id( $view_id );
47
48 6
		if ( ! $view ) {
49
			gravityview()->log->error( 'View does not exist #{view_id}', array( 'view_id' => $view_id ) );
50
			return '';
51
		}
52
53 6
		gravityview()->views->set( $view );
0 ignored issues
show
Documentation introduced by
The property views does not exist on object<GV\Core>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
54
55
		/**
56
		 * When this shortcode is embedded inside a View we can only display it as a directory. There's no other way.
57
		 * Try to detect that we're not embedded to allow edit and single contexts.
58
		 */
59 6
		$is_reembedded = false; // Assume not embedded unless detected otherwise.
60 6
		if ( in_array( get_class( $request ), array( 'GV\Frontend_Request', 'GV\Mock_Request' ) ) ) {
61 6
62
			if ( ( $_view = $request->is_view() ) && $_view->ID !== $view->ID ) {
0 ignored issues
show
Documentation introduced by
The property ID does not exist on object<GV\View>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
63
				$is_reembedded = true;
64
65
			} elseif ( $request->is_entry() ) {
66
				$backtrace = debug_backtrace( ~DEBUG_BACKTRACE_PROVIDE_OBJECT | DEBUG_BACKTRACE_IGNORE_ARGS );
67
				foreach ( array_slice( $backtrace, 1 ) as $call ) {
68
					if ( \GV\Utils::get( $call, 'class' ) == __CLASS__ && \GV\Utils::get( $call, 'function' ) == __FUNCTION__ ) {
69 6
						$is_reembedded = true;
70
					}
71
				}
72
			}
73 6
		}
74
75 6
		/**
76 6
		 * Remove Widgets on a nested embedded View.
77
		 */
78
		if ( $is_reembedded ) {
79
			$view->widgets = new \GV\Widget_Collection();
80
		}
81 6
82 1
		$atts = $this->parse_and_sanitize_atts( $atts );
83
84
		$view->settings->update( $atts );
85 1
		$entries = $view->get_entries( $request );
86 1
87 1
		/**
88 1
		 * Check permissions.
89
		 */
90
		while ( $error = $view->can_render( array( 'shortcode' ), $request ) ) {
91
			if ( ! is_wp_error( $error ) )
0 ignored issues
show
Coding Style Best Practice introduced by
It is generally a best practice to always use braces with control structures.

Adding braces to control structures avoids accidental mistakes as your code changes:

// Without braces (not recommended)
if (true)
    doSomething();

// Recommended
if (true) {
    doSomething();
}
Loading history...
92
				break;
93
94
			switch ( str_replace( 'gravityview/', '', $error->get_error_code() ) ) {
95
				case 'post_password_required':
96
					return get_the_password_form( $view->ID );
0 ignored issues
show
Documentation introduced by
The property ID does not exist on object<GV\View>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
97 1
				case 'no_form_attached':
98 1
					/**
99 1
					 * This View has no data source. There's nothing to show really.
100 1
					 * ...apart from a nice message if the user can do anything about it.
101
					 */
102
					if ( \GVCommon::has_cap( array( 'edit_gravityviews', 'edit_gravityview' ), $view->ID ) ) {
0 ignored issues
show
Documentation introduced by
The property ID does not exist on object<GV\View>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
103
						return __( sprintf( 'This View is not configured properly. Start by <a href="%s">selecting a form</a>.', esc_url( get_edit_post_link( $view->ID, false ) ) ), 'gravityview' );
0 ignored issues
show
Documentation introduced by
The property ID does not exist on object<GV\View>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
introduced by
Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'sprintf'
Loading history...
104 6
					}
105
					break;
106
				case 'no_direct_access':
107
				case 'embed_only':
108
				case 'not_public':
109 6
					return __( 'You are not allowed to view this content.', 'gravityview' );
110 2
			}
111
		}
112
113
		$is_admin_and_can_view = $view->settings->get( 'admin_show_all_statuses' ) && \GVCommon::has_cap('gravityview_moderate_entries', $view->ID );
0 ignored issues
show
Documentation introduced by
The property ID does not exist on object<GV\View>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
114
115 5
		/**
116
		 * View details.
117
		 */
118
		if ( $atts['detail'] ) {
119
			return $this->detail( $view, $entries, $atts );
120
121
		/**
122
		 * Editing a single entry.
123
		 */
124
		} else if ( ! $is_reembedded && ( $entry = $request->is_edit_entry() ) ) {
125
126
			/**
127
			 * When editing an entry don't render multiple views.
128
			 */
129
			if ( ( $selected = \GV\Utils::_GET( 'gvid' ) ) && $view->ID != $selected ) {
0 ignored issues
show
Documentation introduced by
The property ID does not exist on object<GV\View>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
130
				gravityview()->log->notice( 'Entry ID #{entry_id} not rendered because another View ID was passed using `?gvid`: #{selected}', array( 'entry_id' => $entry->ID, 'selected' => $selected ) );
131
				return '';
132
			}
133
134
			if ( $entry['status'] != 'active' ) {
0 ignored issues
show
introduced by
Found "!= '". Use Yoda Condition checks, you must
Loading history...
135
				gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) );
136
				return __( 'You are not allowed to view this content.', 'gravityview' );
137
			}
138
139
			if ( apply_filters( 'gravityview_custom_entry_slug', false ) && $entry->slug != get_query_var( \GV\Entry::get_endpoint_name() ) ) {
140
				gravityview()->log->error( 'Entry ID #{entry_id} was accessed by a bad slug', array( 'entry_id' => $entry->ID ) );
141
				return __( 'You are not allowed to view this content.', 'gravityview' );
142
			}
143
144
			if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) {
145
				if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) )  ) {
146
					gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) );
147
					return __( 'You are not allowed to view this content.', 'gravityview' );
148 5
				}
149
			}
150
151
			$renderer = new \GV\Edit_Entry_Renderer();
152 2
			return $renderer->render( $entry, $view, $request );
153
154
		/**
155
		 * Viewing a single entry.
156 2
		 */
157 1
		} else if ( ! $is_reembedded && ( $entry = $request->is_entry() ) ) {
158 1
			/**
159
			 * When viewing an entry don't render multiple views.
160
			 */
161 2
			if ( ( $selected = \GV\Utils::_GET( 'gvid' ) ) && $view->ID != $selected ) {
0 ignored issues
show
Documentation introduced by
The property ID does not exist on object<GV\View>. Since you implemented __get, maybe consider adding a @property annotation.

Since your code implements the magic getter _get, this function will be called for any read access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

If the property has read access only, you can use the @property-read annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
162 1
				return '';
163 1
			}
164
165
			if ( $entry['status'] != 'active' ) {
0 ignored issues
show
introduced by
Found "!= '". Use Yoda Condition checks, you must
Loading history...
166 2
				gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) );
167 1
				return __( 'You are not allowed to view this content.', 'gravityview' );
168 1
			}
169 1
170
			if ( apply_filters( 'gravityview_custom_entry_slug', false ) && $entry->slug != get_query_var( \GV\Entry::get_endpoint_name() ) ) {
171
				gravityview()->log->error( 'Entry ID #{entry_id} was accessed by a bad slug', array( 'entry_id' => $entry->ID ) );
172
				return __( 'You are not allowed to view this content.', 'gravityview' );
173 2
			}
174
175 2
			if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) {
176 1
				if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) )  ) {
177 1
					gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) );
178
					return __( 'You are not allowed to view this content.', 'gravityview' );
179
				}
180 2
			}
181 2
182
			$error = \GVCommon::check_entry_display( $entry->as_entry() );
183
184
			if( is_wp_error( $error ) ) {
185
				gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing: {message}', array( 'entry_id' => $entry->ID, 'message' => $error->get_error_message() ) );
186
				return __( 'You are not allowed to view this content.', 'gravityview' );
187 4
			}
188
189
			$renderer = new \GV\Entry_Renderer();
190
			return $renderer->render( $entry, $view, $request );
191
192
		/**
193
		 * Just this view.
194
		 */
195
		} else {
196
			if ( $is_reembedded ) {
197
				
198
				// Mock the request with the actual View, not the global one
199 4
				$mock_request = new \GV\Mock_Request();
200 4
				$mock_request->returns['is_view'] = $view;
201
				$mock_request->returns['is_entry'] = $request->is_entry();
202
				$mock_request->returns['is_edit_entry'] = $request->is_edit_entry();
203
				$mock_request->returns['is_search'] = $request->is_search();
204
205
				$request = $mock_request;
206
			}
207
208
			$renderer = new \GV\View_Renderer();
209
			return $renderer->render( $view, $request );
210
		}
211
	}
212
213
	/**
214
	 * Validate attributes passed to the [gravityview] shortcode. Supports {get} Merge Tags values.
215
	 *
216
	 * Attributes passed to the shortcode are compared to registered attributes {@see \GV\View_Settings::defaults}
217
	 * Only attributes that are defined will be allowed through.
218
	 *
219
	 * Then, {get} merge tags are replaced with their $_GET values, if passed
220 5
	 *
221
	 * Then, attributes are sanitized based on the type of setting (number, checkbox, select, radio, text)
222 5
	 *
223
	 * @see \GV\View_Settings::defaults() Only attributes defined in default() are valid to be passed via the shortcode
224 5
	 *
225
	 * @param array $passed_atts Attribute pairs defined to render the View
226
	 *
227 5
	 * @return array Valid and sanitized attribute pairs
228
	 */
229
	private function parse_and_sanitize_atts( $passed_atts ) {
230 5
231
		$defaults = \GV\View_Settings::defaults( true );
232 5
233
		$supported_atts = array_fill_keys( array_keys( $defaults ), '' );
234 5
235
		// Whittle down the attributes to only valid pairs
236
		$filtered_atts = shortcode_atts( $supported_atts, $passed_atts, 'gravityview' );
237 5
238
		// Only keep the passed attributes after making sure that they're valid pairs
239 5
		$filtered_atts = array_intersect_key( (array) $passed_atts, $filtered_atts );
240
241
		$atts = array();
242
243
		foreach( $filtered_atts as $key => $passed_value ) {
244
245
			// Allow using GravityView merge tags in shortcode attributes, like {get} and {created_by}
246 5
			$passed_value = \GravityView_Merge_Tags::replace_variables( $passed_value );
247 5
248 5
			switch( $defaults[ $key ]['type'] ) {
249
250 5
				/**
251
				 * Make sure number fields are numeric.
252
				 * Also, convert mixed number strings to numbers
253 1
				 * @see http://php.net/manual/en/function.is-numeric.php#107326
254
				 */
255
				case 'number':
256
					if( is_numeric( $passed_value ) ) {
257
						$atts[ $key ] = ( $passed_value + 0 );
258
					}
259
					break;
260 1
261 1
				// Checkboxes should be 1 or 0
262
				case 'checkbox':
263
					$atts[ $key ] = gv_empty( $passed_value, true, false ) ? 0 : 1;
264
					break;
265
266
				/**
267
				 * Only allow values that are defined in the settings
268 1
				 */
269
				case 'select':
270 1
				case 'radio':
271 5
					$options = isset( $defaults[ $key ]['choices'] ) ? $defaults[ $key ]['choices'] : $defaults[ $key ]['options'];
272
					if( in_array( $passed_value, array_keys( $options ) ) ) {
273
						$atts[ $key ] = $passed_value;
274
					}
275 5
					break;
276
277 5
				case 'text':
278
				default:
279
					$atts[ $key ] = $passed_value;
280
					break;
281
			}
282
		}
283
284
		$atts['detail'] = \GV\Utils::get( $passed_atts, 'detail', null );
285
286
		return $atts;
287
	}
288
289 1
	/**
290 1
	 * Output view details.
291
	 *
292 1
	 * @param \GV\View $view The View.
293 1
	 * @param \GV\Entry_Collection $entries The calculated entries.
294 1
	 * @param array $atts The shortcode attributes (with defaults).
295 1
	 *
296
	 * @return string The output.
297
	 */
298
	private function detail( $view, $entries, $atts ) {
299
		$output = '';
300
301
		switch ( $key = $atts['detail'] ):
302
			case 'total_entries':
303
				$output = number_format_i18n( $entries->total() );
304
				break;
305
			case 'first_entry':
306
				$output = number_format_i18n( min( $entries->total(), $view->settings->get( 'offset' ) + 1 ) );
307
				break;
308
			case 'last_entry':
309
				$output = number_format_i18n( $view->settings->get( 'page_size' ) + $view->settings->get( 'offset' ) );
310
				break;
311
			case 'page_size':
312
				$output = number_format_i18n( $view->settings->get( $key ) );
313
				break;
314
		endswitch;
315
316
		/**
317 1
		 * @filter `gravityview/shortcode/detail/{$detail}` Filter the detail output returned from `[gravityview detail="$detail"]`
318
		 * @since 1.13
319 1
		 * @param string[in,out] $output Existing output
0 ignored issues
show
Documentation introduced by
The doc-type string[in,out] could not be parsed: Expected "]" at position 2, but found "in". (view supported doc-types)

This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types.

Loading history...
320
		 *
321
		 * @since 2.0.3
322
		 * @param \GV\View $view The view.
323
		 * @param \GV\Entry_Collection $entries The entries.
324
		 * @param array $atts The shortcode atts with defaults.
325
		 */
326
		$output = apply_filters( "gravityview/shortcode/detail/$key", $output, $view );
327
328
		return $output;
329
	}
330
}
331