Code Simplification - Meta Box

This topic has 2 replies, 2 voices, and was last updated 8 years, 6 months ago by 87Pancakes.

Viewing 3 posts - 1 through 3 (of 3 total)

Author

Posts
June 22, 2016 at 8:26 PM #3514
87Pancakes
Participant
Is there a way to simplify this code or is this the correct way to map the data in the frontend?
```
<?php
$field_id = 'hello_bar';
if ( !empty( $settings[$field_id] ) ) {
echo '' . $settings[$field_id] . '';
}
?>
```
Add New Note to this Reply
June 23, 2016 at 8:14 AM #3516
Anh Tran
Keymaster
I think it would be better if you have default values for your settings, and then you don't need to check with if:
```
$default = array();
$settings = get_option( 'setting_id' );
$settings = wp_parse_args( $settings, $default );

// Then
echo $settings[$field_id];
```
Add New Note to this Reply
June 25, 2016 at 4:19 AM #3535

87Pancakes
Participant

Thank-you 🙂

Add New Note to this Reply
Author

Posts

Viewing 3 posts - 1 through 3 (of 3 total)

The topic ‘Code Simplification’ is closed to new replies.