Support » Theme: P2 » PHP 7.3 error – Regular expression needs updating?

  • Resolved prob

    (@prob)


    My googling suggests it may be a regex error, perhaps because php 7.3 PCRE (Perl Compatible Regular Expressions) has been updated to v2.

    Not coming up if I revert to PHP 7.2

    Warning: preg_match_all(): Compilation failed: invalid range in character class at offset 7 in /home2/hearmore/public_html/wp-content/themes/p2/inc/mentions.php on line 77

    The page I need help with: [log in to see the link]

Viewing 10 replies - 1 through 10 (of 10 total)
  • Fotis

    (@fstat)

    Hi there,

    That theme is not being actively developed and that’s why you’re getting these errors when you’re running a newer version of PHP.

    There’s an updated version of that theme that is compatible with PHP 7.3 and has also more features. You can read more about it here:

    https://geto2.com/

    prob

    (@prob)

    Thanks. Installed new P2 theme and O2 plugin. Uninstalled any other plugins that might cause conflict but getting this error.

    I have the old P2 theme still installed but not active.

    Thoughts?

    Warning: “continue” targeting switch is equivalent to “break”. Did you mean to use “continue 2”? in /home2/hearmore/public_html/wp-content/plugins/o2-master/o2.php on line 620
    Warning: Cannot modify header information – headers already sent by (output started at /home2/hearmore/public_html/wp-content/plugins/o2-master/o2.php:620) in /home2/hearmore/public_html/wp-admin/includes/misc.php on line 1196

    Fotis

    (@fstat)

    Hi there,

    That’s just a warning and it should not cause any issues on how the theme works.

    Isn’t the theme working properly for you?

    Reinstalled from scratch and more success this time.

    http://hearmoretunes.com/

    Still not coming up with a mobile layout on mobile devices. Instead the screen is nearly blank. Before I delve into the CSS, is there a mobile / handheld CSS layout provided?

    Also, is this theme going to be developed in the future?

    Moderator James Huff

    (@macmanx)

    Volunteer Moderator

    It’s responsive, so it will adapt for mobile themes.

    I wonder if it’s coming up blank because you have no posts or content displayed on the main page.

    For some reason in your source code on line 195 there is this

    <style>
    		#content {
    			display: none;
    		}
    		</style>

    Which is causing all of your normal content to not display on desktop/mobile versions of the site. Not quite sure where that is coming from though, doesn’t look like its part of the Additional CSS in the Customizer.

    Moderator James Huff

    (@macmanx)

    Volunteer Moderator

    Nice catch @jarretc!

    Weird. I dont believe I have any custom css but will check. Seems unlikely that a plugin is generating this css but I will disable them and see what happens. Is this a theme bug?

    Note that php 7.3 still generates an error. I reverted back to 7.2. If interested I could force this error and share.

    Thanks for help this far.

    Moderator James Huff

    (@macmanx)

    Volunteer Moderator

    It’s coming in right before the end of the head section:

    <style type="text/css" id="syntaxhighlighteranchor"></style>
    <style>
    		#content {
    			display: none;
    		}
    		</style>
    </head>

    So it’s not any custom CSS adding it (and I also doubt it’s related to Syntax Highlighter, I’m just including it because it’s the preceding line).

    Check your copy of the theme’s header.php file, but mostly when I see things injected right before the head section ends, it’s almost always a plugin.

    Resolved by removing plugin.

Viewing 10 replies - 1 through 10 (of 10 total)
  • You must be logged in to reply to this topic.