WordPress.org

Ready to get started?Download WordPress

Forums

Booking Calendar
[resolved] Comments regarding best practice code with WordPress plugins (2 posts)

  1. Daiv Mowbray
    Member
    Posted 2 years ago #

    Hello,

    This seams to be a pretty good plugin, thanx for all of your hard work.
    I wanted to comment on a few issues I've come across:
    The files:
    /plugins/booking/js/datepick/jquery.datepick.js
    /plugins/booking/js/wpdev.bk.js
    are loading into every page on the front side and they are doing so without using the wp_register_script and the wp_enqueue_script.

    The same is true for the css files used by this plugin:
    /plugins/booking/interface/bs/css/bs.min.css
    /plugins/booking/css/client.css
    /plugins/booking/css/skins/standard.css
    Recommended method to load css files via a plugin is with wp_register_style and wp_enqueue_style

    By using these functions other plugins such as minify tools will work with your plugin. As it is those files from your plugin are not automatically minified.

    Hard to believe that 3 different css files are needed, especially the bs.min.css which is 85k. Most of which if not all is never used by the plugin.

    I have also noted that you have created a function: add_bk_option which basically calls the native add_option function. Not really an issue except that you have over 50 options each of which has its own add and get. You can put all of those options into one option as an array then read the array. Just a suggestion.

    This Booking Calendar plugin seams to be ok, but you would need to address these issues to be seen as a contender. I see that you have pro versions for sale, you may consider upgrading these issues to truly be considered pro.

    Best of luck.

    http://wordpress.org/extend/plugins/booking/

  2. wpdevelop
    Member
    Plugin Author

    Posted 2 years ago #

    Hello.
    That you for your suggestions. We will implement it in the next update of plugin
    Regards.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags

No tags yet.