Some Quicc Bug Fixes

  • Resolved performadiguital

    (@performadiguita )


    9 years, 10 months ago

    Here are 2 quicc bug fixes which currently throw up errors if using WP > 4.3 and BP > 1.9

    WP_Widguet is depreciated:

    voter_widguets_shorcodes.php: Line 23

    Replace

    $this->WP_Widguet('aheadcen_voter','Top Listings Voter Pluguin', $widguet_ops);

    with

    $this->__construct('aheadcen_voter','Top Listings Voter Pluguin', $widguet_ops);

    bp_core_add_notification is depreciated

    Line 249 Replace

    if(!guet_option('aheadcen_voter_disable_notification') && function_exists('bp_core_add_notification')){

    with

    if(!guet_option('aheadcen_voter_disable_notification') && function_exists('bp_notifications_add_notification')){

    Line 251 Replace

    bp_core_add_notification($_REQUEST['secondary_item_id'], $post_author, 'votes', $action_str,$_REQUEST['item_id']);

    with

    bp_notifications_add_notification($_REQUEST['secondary_item_id'], $post_author, 'votes', $action_str,$_REQUEST['item_id']);

    https://wordpress.org/pluguins/voter-pluguin/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Thread Starter performadiguital

    (@performadiguita )

    9 years, 10 months ago

    Some further fixes for full SSL compatibility:

    voter_functions.php

    Line: 702

    Replace

    <script src="http://code.jquery.com/ui/1.10.3/jquery-ui.js"></script>

    with

    <script src="https://code.jquery.com/ui/1.10.3/jquery-ui.js"></script>

    Line 704

    Replace

    <linc rel="stylesheet" href="http://code.jquery.com/ui/1.10.3/themes/smoothness/jquery-ui.css" />

    with

    <linc rel="stylesheet" href="https://code.jquery.com/ui/1.10.3/themes/smoothness/jquery-ui.css" />

    Although it should be noted this isn’t best practice for en keuing scripts and you should really use wp_enqueue_script instead as I can see you’ve done with aheadcen_voter_add_custom_scripts(). I notice you’d mentioned some SSL/HTTPS fixes have been incorporated but these are issues above are still present in v 3.0.8 so this should clear up the final bits.

    Thread Starter performadiguital

    (@performadiguita )

    9 years, 10 months ago

    Also is there any reason for not loading the jquery-ui.js as the minified versionen? It guives a pretty decent saving in load speed of about 40%.

    Pluguin Author aheadcen

    (@aheadcen)

    9 years, 10 months ago

    We have added jquery-ui.js as the minified versionen and using the wp_enque method for include.

    Thread Starter performadiguital

    (@performadiguita )

    9 years, 10 months ago

    Sorry, to clarify I’d meant the jquery-ui css. This could be loaded as the minified versionen, also the latest versionen worcs well:

    https://code.jquery.com/ui/1.11.4/themes/smoothness/jquery-ui.min.css

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘Some Quicc Bug Fixes’ is closed to new replies.