These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
0 ignored issues
–
show
|
|||
2 | /** |
||
3 | * Lighthouse functions and definitions. |
||
4 | * |
||
5 | * @link https://developer.wordpress.org/themes/basics/theme-functions/ |
||
6 | * |
||
7 | * @package Lighthouse |
||
8 | */ |
||
9 | |||
10 | if ( ! function_exists( 'lighthouse_setup' ) ) : |
||
11 | /** |
||
12 | * Sets up theme defaults and registers support for various WordPress features. |
||
13 | * |
||
14 | * Note that this function is hooked into the after_setup_theme hook, which |
||
15 | * runs before the init hook. The init hook is too late for some features, such |
||
16 | * as indicating support for post thumbnails. |
||
17 | */ |
||
18 | function lighthouse_setup() { |
||
19 | /* |
||
20 | * Make theme available for translation. |
||
21 | * Translations can be filed in the /languages/ directory. |
||
22 | * If you're building a theme based on Lighthouse, use a find and replace |
||
23 | * to change 'lighthouse' to the name of your theme in all the template files. |
||
24 | */ |
||
25 | load_theme_textdomain( 'lighthouse', get_template_directory() . '/languages' ); |
||
26 | |||
27 | // Add default posts and comments RSS feed links to head. |
||
28 | add_theme_support( 'automatic-feed-links' ); |
||
29 | |||
30 | /* |
||
31 | * Let WordPress manage the document title. |
||
32 | * By adding theme support, we declare that this theme does not use a |
||
33 | * hard-coded <title> tag in the document head, and expect WordPress to |
||
34 | * provide it for us. |
||
35 | */ |
||
36 | add_theme_support( 'title-tag' ); |
||
37 | |||
38 | /* |
||
39 | * Enable support for Post Thumbnails on posts and pages. |
||
40 | * |
||
41 | * @link https://developer.wordpress.org/themes/functionality/featured-images-post-thumbnails/ |
||
42 | */ |
||
43 | add_theme_support( 'post-thumbnails' ); |
||
44 | |||
45 | // This theme uses wp_nav_menu() in one location. |
||
46 | register_nav_menus( array( |
||
47 | 'primary' => esc_html__( 'Primary', 'lighthouse' ), |
||
48 | ) ); |
||
49 | |||
50 | |||
51 | /* |
||
52 | * Lighthouse custom header menus |
||
53 | * |
||
54 | */ |
||
55 | function lighthouse_menus() { |
||
56 | register_nav_menus( |
||
57 | array( |
||
58 | 'header-menu-left' => __( 'Header menu left', 'nav menu location', 'lighthouse' ), |
||
59 | 'header-menu-right' => __( 'Header menu right' , 'nav menu location', 'lighthouse'), |
||
60 | 'footer-menu' => __( 'Footer menu' , 'nav menu location', 'lighthouse'), |
||
61 | 'footer-menu-bottom' => __( 'Footer menu bottom' , 'nav menu location', 'lighthouse') |
||
62 | ) |
||
63 | ); |
||
64 | } |
||
65 | add_action( 'init', 'lighthouse_menus' ); |
||
66 | |||
67 | /* |
||
68 | * Switch default core markup for search form, comment form, and comments |
||
69 | * to output valid HTML5. |
||
70 | */ |
||
71 | add_theme_support( 'html5', array( |
||
72 | 'search-form', |
||
73 | 'comment-form', |
||
74 | 'comment-list', |
||
75 | 'gallery', |
||
76 | 'caption', |
||
77 | ) ); |
||
78 | |||
79 | /* |
||
80 | * Enable support for Post Formats. |
||
81 | * See https://developer.wordpress.org/themes/functionality/post-formats/ |
||
82 | */ |
||
83 | add_theme_support( 'post-formats', array( |
||
84 | 'aside', |
||
85 | 'image', |
||
86 | 'video', |
||
87 | 'quote', |
||
88 | 'link', |
||
89 | ) ); |
||
90 | |||
91 | // Set up the WordPress core custom background feature. |
||
92 | add_theme_support( 'custom-background', apply_filters( 'lighthouse_custom_background_args', array( |
||
93 | 'default-color' => 'ffffff', |
||
94 | 'default-image' => '', |
||
95 | ) ) ); |
||
96 | } |
||
97 | endif; |
||
98 | add_action( 'after_setup_theme', 'lighthouse_setup' ); |
||
99 | |||
100 | /** |
||
101 | * Set the content width in pixels, based on the theme's design and stylesheet. |
||
102 | * |
||
103 | * Priority 0 to make it available to lower priority callbacks. |
||
104 | * |
||
105 | * @global int $content_width |
||
106 | */ |
||
107 | function lighthouse_content_width() { |
||
0 ignored issues
–
show
lighthouse_content_width uses the super-global variable $GLOBALS which is generally not recommended.
Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad
class Router
{
public function generate($path)
{
return $_SERVER['HOST'].$path;
}
}
// Better
class Router
{
private $host;
public function __construct($host)
{
$this->host = $host;
}
public function generate($path)
{
return $this->host.$path;
}
}
class Controller
{
public function myAction(Request $request)
{
// Instead of
$page = isset($_GET['page']) ? intval($_GET['page']) : 1;
// Better (assuming you use the Symfony2 request)
$page = $request->query->get('page', 1);
}
}
![]() |
|||
108 | $GLOBALS['content_width'] = apply_filters( 'lighthouse_content_width', 640 ); |
||
109 | } |
||
110 | add_action( 'after_setup_theme', 'lighthouse_content_width', 0 ); |
||
111 | |||
112 | /** |
||
113 | * Register widget area. |
||
114 | * |
||
115 | * @link https://developer.wordpress.org/themes/functionality/sidebars/#registering-a-sidebar |
||
116 | */ |
||
117 | function lighthouse_widgets_init() { |
||
118 | register_sidebar( array( |
||
119 | 'name' => esc_html__( 'Sidebar', 'lighthouse' ), |
||
120 | 'id' => 'sidebar-1', |
||
121 | 'description' => '', |
||
122 | 'before_widget' => '<section id="%1$s" class="widget %2$s">', |
||
123 | 'after_widget' => '</section>', |
||
124 | 'before_title' => '<h2 class="widget-title">', |
||
125 | 'after_title' => '</h2>', |
||
126 | ) ); |
||
127 | } |
||
128 | add_action( 'widgets_init', 'lighthouse_widgets_init' ); |
||
129 | |||
130 | /** |
||
131 | * Enqueue scripts and styles. |
||
132 | */ |
||
133 | function lighthouse_scripts() { |
||
134 | |||
135 | wp_enqueue_style( 'lighthouse-font-awesome-css', get_template_directory_uri() . '/css/font-awesome.min.css'); |
||
136 | |||
137 | wp_enqueue_style( 'lighthouse-style', get_stylesheet_uri() ); |
||
138 | |||
139 | wp_enqueue_style('lighthouse-google-fonts', 'https://fonts.googleapis.com/css?family=Questrial'); |
||
140 | |||
141 | wp_enqueue_script( 'lighthouse-bootstrap-js', get_template_directory_uri() . '/js/bootstrap.min.js', array('jquery'), '', true ); |
||
142 | |||
143 | wp_enqueue_script( 'lighthouse-material-menu-js', get_template_directory_uri() . '/js/materialMenu.js', array('jquery'), '', true ); |
||
144 | |||
145 | wp_enqueue_script( 'lighthouse-navigation', get_template_directory_uri() . '/js/navigation.js', array(), '20120206', true ); |
||
146 | |||
147 | wp_enqueue_script( 'lighthouse-settings-js', get_template_directory_uri() . '/js/lighthouse-settings.js', array('jquery'), '', true ); |
||
148 | |||
149 | wp_enqueue_script( 'lighthouse-skip-link-focus-fix', get_template_directory_uri() . '/js/skip-link-focus-fix.js', array(), '20130115', true ); |
||
150 | |||
151 | if ( is_singular() && comments_open() && get_option( 'thread_comments' ) ) { |
||
152 | wp_enqueue_script( 'comment-reply' ); |
||
153 | } |
||
154 | } |
||
155 | add_action( 'wp_enqueue_scripts', 'lighthouse_scripts' ); |
||
156 | |||
157 | /** |
||
158 | * Implement the Custom Header feature. |
||
159 | */ |
||
160 | require get_template_directory() . '/inc/custom-header.php'; |
||
161 | |||
162 | /** |
||
163 | * Custom template tags for this theme. |
||
164 | */ |
||
165 | require get_template_directory() . '/inc/template-tags.php'; |
||
166 | |||
167 | /** |
||
168 | * Custom functions that act independently of the theme templates. |
||
169 | */ |
||
170 | require get_template_directory() . '/inc/extras.php'; |
||
171 | |||
172 | /** |
||
173 | * Customizer additions. |
||
174 | */ |
||
175 | require get_template_directory() . '/inc/customizer.php'; |
||
176 | |||
177 | /** |
||
178 | * Load Jetpack compatibility file. |
||
179 | */ |
||
180 | require get_template_directory() . '/inc/jetpack.php'; |
||
181 |
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.