| Conditions | 11 | 
| Paths | 1024 | 
| Total Lines | 152 | 
| Code Lines | 83 | 
| Lines | 0 | 
| Ratio | 0 % | 
| Changes | 4 | ||
| Bugs | 0 | Features | 0 | 
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php  | 
            ||
| 118 | function ppp_display_social() { | 
            ||
| 119 | do_action( 'ppp_social_settings_pre_form' );  | 
            ||
| 120 | |||
| 121 | $ppp_share_settings = get_option( 'ppp_share_settings' );  | 
            ||
| 122 | ?>  | 
            ||
| 123 | <div class="wrap">  | 
            ||
| 124 | <div id="icon-options-general" class="icon32"></div><h1><?php _e( 'Post Promoter Pro - Social Settings', 'ppp-txt' ); ?></h1>  | 
            ||
| 125 | <form method="post" action="options.php">  | 
            ||
| 126 | <?php wp_nonce_field( 'ppp-share-settings' ); ?>  | 
            ||
| 127 | <h3><?php _e( 'Social Media Accounts', 'ppp-txt' ); ?></h3>  | 
            ||
| 128 | <?php  | 
            ||
| 129 | require_once PPP_PATH . 'includes/admin/class-accounts-table.php';  | 
            ||
| 130 | |||
| 131 | $accounts_table = new PPP_Accounts_Table();  | 
            ||
| 132 | $accounts_table->prepare_items();  | 
            ||
| 133 | |||
| 134 | $accounts_table->display();  | 
            ||
| 135 | ?>  | 
            ||
| 136 | <table class="form-table">  | 
            ||
| 137 | <?php $analytics_option = isset( $ppp_share_settings['analytics'] ) ? $ppp_share_settings['analytics'] : 0; ?>  | 
            ||
| 138 | <tr valign="top">  | 
            ||
| 139 | <th scope="row" valign="top">  | 
            ||
| 140 | <?php _e( 'Analytics', 'ppp-txt' ); ?></span>  | 
            ||
| 141 | </th>  | 
            ||
| 142 | <td id="ppp-analytics-options">  | 
            ||
| 143 | <p>  | 
            ||
| 144 | <input id="ppp_no_tracking"  | 
            ||
| 145 | name="ppp_share_settings[analytics]"  | 
            ||
| 146 | type="radio"  | 
            ||
| 147 | value="none"  | 
            ||
| 148 | <?php checked( 'none', $analytics_option, true ); ?>  | 
            ||
| 149 | /> <label for="ppp_no_tracking"><?php _e( 'None', 'ppp-txt' ); ?></label>  | 
            ||
| 150 | </p>  | 
            ||
| 151 | <br />  | 
            ||
| 152 | <p>  | 
            ||
| 153 | <input id="ppp_unique_links"  | 
            ||
| 154 | name="ppp_share_settings[analytics]"  | 
            ||
| 155 | type="radio"  | 
            ||
| 156 | value="unique_links"  | 
            ||
| 157 | <?php checked( 'unique_links', $analytics_option, true ); ?>  | 
            ||
| 158 | /> <label for="ppp_unique_links"><?php _e( 'Simple Tracking', 'ppp-txt' ); ?></label><br />  | 
            ||
| 159 | <small><?php _e( 'Appends a query string to shared links for analytics.', 'ppp-txt' ); ?></small>  | 
            ||
| 160 | </p>  | 
            ||
| 161 | <br />  | 
            ||
| 162 | <p>  | 
            ||
| 163 | <input id="ppp_ga_tags"  | 
            ||
| 164 | name="ppp_share_settings[analytics]"  | 
            ||
| 165 | type="radio"  | 
            ||
| 166 | value="google_analytics"  | 
            ||
| 167 | <?php checked( 'google_analytics', $analytics_option, true ); ?>  | 
            ||
| 168 | /> <label for="ppp_ga_tags"><?php _e( 'Google Analytics Tags', 'ppp-txt' ); ?></label><br />  | 
            ||
| 169 | <small><?php _e( 'Results can be seen in the Acquisition Menu under "Campaigns"', 'ppp-txt' ); ?></small>  | 
            ||
| 170 | </p>  | 
            ||
| 171 | <?php do_action( 'ppp-settings-analytics-radio' ); ?>  | 
            ||
| 172 | <p id="ppp-link-example">  | 
            ||
| 173 | <hr />  | 
            ||
| 174 | <small><?php _e( 'Here is an example of what your link will look like', 'ppp-txt' ); ?>: <br />  | 
            ||
| 175 | <?php $post = wp_get_recent_posts( array( 'numberposts' => 1 ) ); ?>  | 
            ||
| 176 | <?php if( count( $post ) > 0 ): ?>  | 
            ||
| 177 | <code><?php echo ppp_generate_link( $post[0]['ID'], 'sharedate_1_' . $post[0]['ID'], false ); ?></code></small>  | 
            ||
| 178 | <?php else: ?>  | 
            ||
| 179 | <em><?php _e( 'No posts available to generate link from.', 'ppp-txt' ); ?></em>  | 
            ||
| 180 | <?php endif; ?>  | 
            ||
| 181 | </p>  | 
            ||
| 182 | </td>  | 
            ||
| 183 | </tr>  | 
            ||
| 184 | |||
| 185 | <?php  | 
            ||
| 186 | $tw_sop = ! empty( $ppp_share_settings['twitter']['share_on_publish'] ) ? true : false;  | 
            ||
| 187 | $fb_sop = ! empty( $ppp_share_settings['facebook']['share_on_publish'] ) ? true : false;  | 
            ||
| 188 | $li_sop = ! empty( $ppp_share_settings['linkedin']['share_on_publish'] ) ? true : false;  | 
            ||
| 189 | ?>  | 
            ||
| 190 | <tr valign="top">  | 
            ||
| 191 | <th scope="row" valign="top">  | 
            ||
| 192 | <?php _e( 'Share on Publish Defaults', 'ppp-txt' ); ?></span><br />  | 
            ||
| 193 | <small><em><?php _e( 'Enabled sharing on publish by default', 'ppp-txt' ); ?></em></small>  | 
            ||
| 194 | </th>  | 
            ||
| 195 | <td id="ppp-share-on-publish-wrapper">  | 
            ||
| 196 | <?php if ( ppp_twitter_enabled() ) : ?>  | 
            ||
| 197 | <p>  | 
            ||
| 198 | <input type="checkbox" id="twitter-share-on-publish" value="1" <?php checked( true, $tw_sop, true ); ?> name="ppp_share_settings[twitter][share_on_publish]" />  | 
            ||
| 199 | <label for="twitter-share-on-publish"><?php _e( 'Twitter', 'ppp-txt' ); ?></label>  | 
            ||
| 200 | </p>  | 
            ||
| 201 | <?php endif; ?>  | 
            ||
| 202 | |||
| 203 | <?php if ( ppp_facebook_enabled() ) : ?>  | 
            ||
| 204 | <p>  | 
            ||
| 205 | <input type="checkbox" id="facebook-share-on-publish" value="1" <?php checked( true, $fb_sop, true ); ?> name="ppp_share_settings[facebook][share_on_publish]" />  | 
            ||
| 206 | <label for="facebook-share-on-publish"><?php _e( 'Facebook', 'ppp-txt' ); ?></label>  | 
            ||
| 207 | </p>  | 
            ||
| 208 | <?php endif; ?>  | 
            ||
| 209 | |||
| 210 | <?php if ( ppp_linkedin_enabled() ) : ?>  | 
            ||
| 211 | <p>  | 
            ||
| 212 | <input type="checkbox" id="linkedin-share-on-publish" value="1" <?php checked( true, $li_sop, true ); ?> name="ppp_share_settings[linkedin][share_on_publish]" />  | 
            ||
| 213 | <label for="linkedin-share-on-publish"><?php _e( 'LinkedIn', 'ppp-txt' ); ?></label>  | 
            ||
| 214 | </p>  | 
            ||
| 215 | <?php endif; ?>  | 
            ||
| 216 | </td>  | 
            ||
| 217 | </tr>  | 
            ||
| 218 | |||
| 219 | <?php $twitter_cards_enabled = ppp_tw_cards_enabled(); ?>  | 
            ||
| 220 | <tr valign="top">  | 
            ||
| 221 | <th scope="row" valign="top">  | 
            ||
| 222 | <?php _e( 'Twitter Settings', 'ppp-txt' ); ?></span>  | 
            ||
| 223 | </th>  | 
            ||
| 224 | <td id="ppp-twitter-cards-wrapper">  | 
            ||
| 225 | <p>  | 
            ||
| 226 | <input id="ppp-twitter-cards"  | 
            ||
| 227 | name="ppp_share_settings[twitter][cards_enabled]"  | 
            ||
| 228 | type="checkbox"  | 
            ||
| 229 | value="1"  | 
            ||
| 230 | <?php checked( true, $twitter_cards_enabled, true ); ?>  | 
            ||
| 231 | /> <label for="ppp-twitter-cards"><?php _e( 'Enable Twitter Cards', 'ppp-txt' ); ?></label>  | 
            ||
| 232 | </p>  | 
            ||
| 233 | </td>  | 
            ||
| 234 | </tr>  | 
            ||
| 235 | |||
| 236 | <?php  | 
            ||
| 237 | $shortener = isset( $ppp_share_settings['shortener'] ) ? $ppp_share_settings['shortener'] : false;  | 
            ||
| 238 | ?>  | 
            ||
| 239 | <tr valign="top">  | 
            ||
| 240 | <th scope="row" valign="top">  | 
            ||
| 241 | <?php _e( 'URL Shortener', 'ppp-txt' ); ?></span>  | 
            ||
| 242 | </th>  | 
            ||
| 243 | <td id="ppp-shortener-options">  | 
            ||
| 244 | <p>  | 
            ||
| 245 | <select name="ppp_share_settings[shortener]">  | 
            ||
| 246 | <option value="-1"><?php _e( 'Select a Service', 'ppp-txt' ); ?></option>  | 
            ||
| 247 | <?php do_action( 'ppp_url_shorteners', $shortener ); ?>  | 
            ||
| 248 | </select>  | 
            ||
| 249 | </p>  | 
            ||
| 250 | <?php if ( $shortener ) : ?>  | 
            ||
| 251 | <?php do_action( 'ppp_shortener_settings-' . $shortener ); ?>  | 
            ||
| 252 | <?php endif; ?>  | 
            ||
| 253 | </td>  | 
            ||
| 254 | </tr>  | 
            ||
| 255 | |||
| 256 | <?php settings_fields( 'ppp-share-settings' ); ?>  | 
            ||
| 257 | |||
| 258 | <input type="hidden" name="action" value="update" />  | 
            ||
| 259 | <input type="hidden" name="page_options" value="ppp_share_settings,ppp_social_settings" />  | 
            ||
| 260 | |||
| 261 | |||
| 262 | </table>  | 
            ||
| 263 | |||
| 264 | <input type="submit" class="button-primary" value="<?php _e( 'Save Changes', 'ppp-txt' ) ?>" />  | 
            ||
| 265 | |||
| 266 | </form>  | 
            ||
| 267 | </div>  | 
            ||
| 268 | <?php  | 
            ||
| 269 | }  | 
            ||
| 270 | |||
| 395 |