Skip to:
Content

BuddyPress.org

Changeset 5690


Ignore:
Timestamp:
02/10/2012 01:35:34 AM (13 years ago)
Author:
johnjamesjacoby
Message:

Remove unneeded globals and clean up some code in Settings component. See #3989.

Location:
trunk/bp-settings
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/bp-settings/bp-settings-actions.php

    r5588 r5690  
    11<?php
     2
     3/**
     4 * BuddyPress Settings Actions
     5 *
     6 * @todo split actions into separate screen functions
     7 * @package BuddyPress
     8 * @subpackage SettingsActions
     9 */
     10
    211// Exit if accessed directly
    312if ( !defined( 'ABSPATH' ) ) exit;
    … …  
    124133
    125134function bp_core_screen_notification_settings() {
    126     global $bp;
    127135
    128136    if ( bp_action_variables() ) {
    … …  
    154162
    155163function bp_core_screen_delete_account() {
    156     global $bp;
    157164
    158165    if ( bp_action_variables() ) {

  • trunk/bp-settings/bp-settings-functions.php

    r4826 r5690  
    11<?php
     2
     3/**
     4 * BuddyPress Settings Functions
     5 *
     6 * @package BuddyPress
     7 * @subpackage SettingsFunctions
     8 */
     9
    210// Exit if accessed directly
    311if ( !defined( 'ABSPATH' ) ) exit;
     12
    413?>

  • trunk/bp-settings/bp-settings-loader.php

    r5329 r5690  
    11<?php
     2
     3/**
     4 * BuddyPress Settings Loader
     5 *
     6 * @package BuddyPress
     7 * @subpackage SettingsLoader
     8 */
     9
    210// Exit if accessed directly
    311if ( !defined( 'ABSPATH' ) ) exit;
    … …  
    4553     */
    4654    function setup_globals() {
    47         global $bp;
    4855
    4956        // Define a slug, if necessary
    … …  
    6774    function setup_nav() {
    6875        global $bp;
     76
     77        // Define local variable
     78        $sub_nav = array();
    6979
    7080        // Add the settings navigation item

  • trunk/bp-settings/bp-settings-screens.php

    r4826 r5690  
    11<?php
     2
     3/**
     4 * BuddyPress Settings Screens
     5 *
     6 * @package BuddyPress
     7 * @subpackage SettingsFunctions
     8 */
     9
    210// Exit if accessed directly
    311if ( !defined( 'ABSPATH' ) ) exit;
     12
    413?>

  • trunk/bp-settings/bp-settings-template.php

    r4961 r5690  
    11<?php
     2
    23/**
    34 * BuddyPress Settings Template Functions
    45 *
    56 * @package BuddyPress
    6  * @subpackage Settings Template
     7 * @subpackage SettingsTemplate
    78 */
    89
    … …  
    1415 *
    1516 * @package BuddyPress
    16  * @subpackage Settings Template
     17 * @subpackage SettingsTemplate
    1718 * @since 1.5
    1819 *
    … …  
    2627     *
    2728     * @package BuddyPress
    28      * @subpackage Settings Template
     29     * @subpackage SettingsTemplate
    2930     * @since 1.5
    3031     */
    … …  
    3839 *
    3940 * @package BuddyPress
    40  * @subpackage Settings Template
     41 * @subpackage SettingsTemplate
    4142 * @since 1.5
    4243 *
    … …  
    5051     *
    5152     * @package BuddyPress
    52      * @subpackage Settings Template
     53     * @subpackage SettingsTemplate
    5354     * @since 1.5
    5455     */
Note: See TracChangeset for help on using the changeset viewer.