Show Sidebar Log in
Commons In A Box Logo
  • Home
  • About
    • Project Team
    • Logos and Graphics
  • Showcase
    • CBOX Classic Showcase
    • CBOX OpenLab Showcase
  • Demo
  • Get Started
  • Documentation
    • Technical Guide
    • CBOX Classic Guide
    • CBOX OpenLab Guide
  • Support Forums
    • CBOX Classic
    • CBOX OpenLab
    • Developers Forum
  • News

Group Admins

  • Profile picture of Scott Voth

CBOX Classic Support

Public Group active 3 months, 4 weeks ago

This group provides support for Commons In A Box Classic, our original software for community-building. Register for an account or log in to commonsinabox.org, then join the group and post your question here.

Wanted to check on some PHP sniffer errors.

  • This topic has 2 replies, 2 voices, and was last updated 8 years, 4 months ago by Troy Welch.
Viewing 3 posts - 1 through 3 (of 3 total)
  • Author
    Posts
  • January 4, 2017 at 12:05 pm #6176
    Troy Welch
    Participant

    Hi, Happy New Year to the Commons in a Box Team

    We are moving our development install to a production server and in preparation for a move to PHP 7 I ran the PHP Compatibility Checker plugin.

    There were several warnings and a small few errors. I just wanted to run the errors past you guys to see if they are anything to worry about. The last error looks like it might be a missing ; but my php is ‘developing’ and I don’t know your code base as well as you guys. (obviously)

    <p class=”p1″>Name: BuddyPress Docs</p>
    <p class=”p1″>FILE: /var/www/html/wp-content/plugins/buddypress-docs/includes/query-builder.php</p>
    <p class=”p1″>————————————————————————————————————————-</p>
    <p class=”p1″>FOUND 1 ERROR AFFECTING 1 LINE</p>
    <p class=”p1″>————————————————————————————————————————-</p>
    <p class=”p1″>521 | ERROR | ‘clone’ is a reserved keyword introduced in PHP version 5.0 and cannot be invoked as a function (T_CLONE)</p>
    <p class=”p1″>————————————————————————————————————————-</p>
    <p class=”p1″>Name: Commons In A Box</p>
    <p class=”p1″>FILE: /var/www/html/wp-content/themes/cbox-theme/engine/ICE/lib/textile/classTextile.php</p>
    <p class=”p1″>—————————————————————————————————————–</p>
    <p class=”p1″>FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES</p>
    <p class=”p1″>—————————————————————————————————————–</p>
    <p class=”p1″><span class=”Apple-converted-space”>  </span>364 | WARNING | Use of deprecated PHP4 style class constructor is not supported since PHP 7.</p>
    <p class=”p1″><span class=”Apple-converted-space”>  </span>617 | ERROR <span class=”Apple-converted-space”>  </span>| Function split() is deprecated since PHP 5.3 and removed since PHP 7.0; use preg_split instead</p>
    <p class=”p1″>1531 | ERROR <span class=”Apple-converted-space”>  </span>| preg_replace() – /e modifier is deprecated since PHP 5.5 and removed since PHP 7.0</p>
    <p class=”p1″>—————————————————————————————————————–</p>
    <p class=”p1″></p>
    <p class=”p1″>FILE: /var/www/html/wp-content/themes/cbox-theme/engine/includes/buddypress/bp-widgets.php</p>
    <p class=”p1″>——————————————————————————————</p>
    <p class=”p1″>FOUND 1 ERROR AFFECTING 1 LINE</p>
    <p class=”p1″>——————————————————————————————</p>
    <p class=”p1″>43 | ERROR | Using a variable argument on break or continue is forbidden since PHP 5.4</p>

    <hr />

    <p class=”p1″></p>
    <p class=”p1″>Thanks,</p>
    <p class=”p1″>Troy</p>

    January 5, 2017 at 11:38 am #6177
    Ray
    Keymaster

    Thanks for looking into PHP7 compatibility with CBOX, Troy.

    I’ve run cbox-theme and buddypress-docs through a script called php7cc and only cbox-theme has major errors.

    I’ve opened an issue for cbox-theme here:
    https://github.com/cuny-academic-commons/cbox-theme/issues/253

    buddypress-docs has a potential warning with the following:

    File: wp-content\plugins\buddypress-docs\includes\addon-folders.php     
    > Line 859: Possible array element creation during by-reference assignment                                     
        $ref[$folder->ID] =& $tree[$folder->ID];                                                                   
    > Line 878: Possible array element creation during by-reference assignment                                     
        $ref[$folder->ID] =& $ref[$folder->post_parent]->children[$folder->ID];

    I’ll let @cbox_admin comment on that.

    January 5, 2017 at 12:05 pm #6178
    Troy Welch
    Participant

    Thanks for the follow-up Ray. We are using the CBox theme with our install and buddypress-docs also figures fairly prominently in our use cases.

    We are looking to move our project to a production server for testing next week for an end of the month launch. Would you suggest we hang back with PHP 5.6 for the time being? We did build on our dev server with 5.6, it might be more prudent.

     

    Thanks again,

    Troy

  • Author
    Posts
Viewing 3 posts - 1 through 3 (of 3 total)
  • You must be logged in to reply to this topic.
Log In
Group logo of CBOX Classic Support
  • Home
  • Forum
  • Announcements
  • Docs
  • Members 287
  • Send Invites

Groups

Newest | Active | Popular | Alphabetical
  • Group logo of CBOX Classic Support
    CBOX Classic Support
    287 members
  • Group logo of CBOX Pioneers
    CBOX Pioneers
    71 members
  • Group logo of CBOX Developers
    CBOX Developers
    40 members
  • Group logo of CBOX OpenLab Support
    CBOX OpenLab Support
    22 members
  • Group logo of CBOX-OL Testing Partners
    CBOX-OL Testing Partners
    12 members

CBOX has its roots in the CUNY Academic Commons, which in turn was made possible through funding from The City University of New York itself.

CUNY Logo

CUNY Academic Commons Logo

City Tech logo

The Commons In A Box was made possible through the generous support of the Alfred P. Sloan Foundation.

Alfred P. Sloan Foundation Logo

NEH Logo

The CUNY Graduate Center has directly contributed to the CUNY Academic Commons, housing the project since its inception, and has contributed to CBOX through its GC Digital Initiatives.

CUNY Graduate Center Logo

CUNY Graduate Center Digital Initiatives Logo

Powered by Commons In A Box
css.php
Skip to toolbar
  • About WordPress
    • WordPress.org
    • Documentation
    • Learn WordPress
    • Support
    • Feedback
  • Log In
  • Register