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

    (@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

    Thread Starter madvic

    (@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

    This is not a support forum. Please post in https://www.ads-software.com/support/theme/bootstrap-ultimate

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

    Theme Author eminozlem

    (@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

    (@eminozlem)

    Also I think I found the reason of those errors see : https://www.ads-software.com/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

    Thread Starter madvic

    (@madvic)

    OK, sorry.
    I test that.

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