WordPress.org

Support

Support » Reviews » many parameters

many parameters

Viewing 6 replies - 1 through 6 (of 6 total)
  • Theme Author eminozlem
    Member

    @eminozlem

    I cant possibly test the theme with every wp or php version, there are bound to be some errors on some. I was planning on doing some extensive tests though.
    But, does any of those errors prevent your system working, cause a crash or something ? If not -which I believe is the case-, I dont think a 1 star is a fair review.

    Can you post the error messages together with your wp & php version so that I can fix them ? Also like I said, please change your vote into a more reasonable one

    madvic
    Participant

    @madvic

    OK,
    in 5.4.16 php version.
    This error block the admin, I can’t navigate in the menu. For me it’s not good for utilisation.
    Correct this error and I can test this theme and I will change my note.

    Strict standards: Declaration of eo_Walker_Nav_Menu::start_lvl() should be compatible with Walker::start_lvl(&$output, $depth = 0, $args = Array) in D:\wamp\www\wordpress381\wp-content\themes\bootstrap-ultimate\inc\core\menu_nav_walker.php on line 89

    Strict standards: Declaration of eo_Walker_Nav_Menu::start_el() should be compatible with Walker::start_el(&$output, $object, $depth = 0, $args = Array, $current_object_id = 0) in D:\wamp\www\wordpress381\wp-content\themes\bootstrap-ultimate\inc\core\menu_nav_walker.php on line 89

    Strict standards: Declaration of eo_mobile_Walker_Nav_Menu::start_lvl() should be compatible with Walker::start_lvl(&$output, $depth = 0, $args = Array) in D:\wamp\www\wordpress381\wp-content\themes\bootstrap-ultimate\inc\core\mobile_menu_nav_walker.php on line 109

    Strict standards: Declaration of eo_mobile_Walker_Nav_Menu::start_el() should be compatible with Walker::start_el(&$output, $object, $depth = 0, $args = Array, $current_object_id = 0) in D:\wamp\www\wordpress381\wp-content\themes\bootstrap-ultimate\inc\core\mobile_menu_nav_walker.php on line 109

    esmi
    Forum Moderator

    @esmi

    This is not a support forum. Please post in http://wordpress.org/support/theme/bootstrap-ultimate

    And next time, try posting a support topic before you post a review. Try reading:
    http://pippinsplugins.com/how-to-leave-a-good-bad-review/
    http://chrislema.com/theres-wrong-way-give-plugin-feedback-wordpress-org/

    Theme Author eminozlem
    Member

    @eminozlem

    Even so, it’s not fair to rate 1 star for PHP warnings. Like I said, we can not possibly test it under every condition and all themes / plugins are bound to have some warnings here and there.
    So I have to ask you to read the links esmi posted and change your rating to a fair one.

    I’ve only tested the theme in PHP 5.3.x, I will check it in 5.4 and fix those warnings in the next release. But they shouldnt prevent it from functioning. How can you not “navigate in the menu” can you expand on that ?

    For now you can:
    1- Disable warnings temprorarily.
    2- Go to theme settings > “Customize” tab and turn OFF “Custom markup”.
    That will disable the custom menu walker and you should be fine.

    Theme Author eminozlem
    Member

    @eminozlem

    Also I think I found the reason of those errors see : http://wordpress.org/support/topic/strict-standards-declaration-of-should-be-compatible-with#post-4757071

    Like mentioned there, adding an $id=0 to the mentioned lines should fix the error.
    function start_el( &$output, $item, $depth, $args ) {
    to
    function start_el( &$output, $item, $depth, $args, $id=0 ) {

    I’ll test the theme entirely in php 5.4 and fix any warnings in the next release.

    And next time post any support requests to support forums

    madvic
    Participant

    @madvic

    OK, sorry.
    I test that.

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘many parameters’ is closed to new replies.