functions.php Bug in theme

Hi,
functions.php is the only theme file that does not require a closing php tag <?php ?>

You have some bad php code the last line ?php>, a closing tag would be ?>, but not required in this file!

Does the plugin say you need to add this code to functions.php, if so which plugin is it?

It should be more like this:

<?php
/* Theme Mega Menu. */
 function register_my_menus() {
 register_nav_menus(
	array( 'header-menu' => __( 'Header Menu' ) )
	);
 }
 add_action( 'init', 'register_my_menus' );

Do you really need a plugin or is this something you can do without, adding the styles etc, here is my example child theme?

HTH

David

Hi Esmi,
The OP did have a bad tag in the code.

Closing php tag is not a php requirement but a WordPress Coding Standard

Quote from Nancin

It is the WordPress coding standard to include closing PHP tags on files that are not expected to be edited.

That excludes a theme’s functions.php and wp-config.php.

The ‘headers already sent’ argument only works for those files. All core files should not be edited, and should have closing tags.

WordPress Codex headers already sent errors

1. Download the file mentioned in the error message via FTP or the file manager provided in your host’s control panel.
2. Open that file in a plain text editor (NOT MS Word or similar. Notepad or BBEdit are fine).
3. Check that the very first characters are <?php
4. Check that the very last characters are ?>

I learnt php via WordPress and do not use it outside of WordPress, so that is where I picked up the advice in the past, but it is good to know that they are not required Per se.

Cheers

David

Cool,
It seems a standard that most plugin and theme developers seem to use, I did not know it was not a requirement, but it seems to be a good practice.

If it is a WordPress coding standard then themes and plugins should really follow suit, for code consistancy when other developers are starting out and looking at code.

Back to the topic ?php> is not valid in this case.

David 🙂

I am so glad I found this thread as this just happened to me moments ago, I was pulling my hair out! After reading here I yanked the closing tag and my Update and Publish buttons in admin worked like they’re supposed to. Having done just enough PHP outside of WP to be dangerous I was totally unaware that you could leave out closing tags.

Gotta love WordPress.org Support Forums;)