Skip to content

Conversation

@timothyjensen
Copy link
Contributor

Just some minor formatting adjustments to bring the snippets inline with WordPress Coding Standards.

add_filter( "sanitize_option_genesis-cpt-archive-settings-{$this->post_type}", array(
$this,
'add_sanitized_values',
), 999 );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure how the sniffer missed this, but callback arrays should not be split to multiple lines.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, I'll revert this.

// Get it started
// myprefix_genesis_cpt_settings( 'custom-post-type-slug' );
// Get it started.
//myprefix_genesis_cpt_settings( 'custom-post-type-slug' );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a fan of //comments w/o space

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, I'll revert this change.

@jtsternberg
Copy link
Member

Thank you sir!

@jtsternberg jtsternberg merged commit e7b9d49 into CMB2:master Apr 13, 2017
@timothyjensen
Copy link
Contributor Author

No, thank you! This will be a great resource for Genesis folks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants