WordPress.org

Forums

Bootstrap Ultimate
many parameters (7 posts)

  1. madvic
    Member
    Posted 1 year ago #

    A theme with a lot of botstrap parameters and thinking for mobile.

  2. eminozlem
    Member
    Theme Author

    Posted 1 year ago #

    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

  3. madvic
    Member
    Posted 1 year ago #

    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

  4. esmi
    Forum Moderator
    Posted 1 year ago #

    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/

  5. eminozlem
    Member
    Theme Author

    Posted 1 year ago #

    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.

  6. eminozlem
    Member
    Theme Author

    Posted 1 year ago #

    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

  7. madvic
    Member
    Posted 1 year ago #

    OK, sorry.
    I test that.

Topic Closed

This topic has been closed to new replies.

About this Theme

About this Topic

Tags

No tags yet.