Skip to content

Commit

Permalink
Merge pull request #82 from DripEmail/eci-2086-fix-deprecation-warnings
Browse files Browse the repository at this point in the history
ECI-2086 Fix deprecation warnings in PHP >= 8.2
  • Loading branch information
diegodrip authored Oct 25, 2024
2 parents 3e9d766 + 09e098d commit 8b4a77e
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 8 deletions.
4 changes: 2 additions & 2 deletions devtools_wp/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FROM wordpress:6.6.2-php8.1-apache
FROM wordpress:6.6.2-php8.3-apache

RUN curl https://raw.githubusercontent.com/wp-cli/builds/gh-pages/phar/wp-cli.phar -o /usr/local/bin/wp && chmod +x /usr/local/bin/wp
RUN apt-get update && apt-get install -y less unzip vim wget vim-tiny net-tools

RUN curl https://downloads.wordpress.org/plugin/woocommerce.9.3.1.zip -o /usr/src/woocommerce.zip && cd /usr/src/wordpress/wp-content/plugins && unzip /usr/src/woocommerce.zip
RUN curl https://downloads.wordpress.org/plugin/woocommerce.9.3.3.zip -o /usr/src/woocommerce.zip && cd /usr/src/wordpress/wp-content/plugins && unzip /usr/src/woocommerce.zip
4 changes: 2 additions & 2 deletions drip.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,13 @@
Plugin Name: Drip for WooCommerce
Plugin URI: https://github.com/DripEmail/drip-woocommerce
Description: A WordPress plugin to connect to Drip's WooCommerce integration
Version: 1.1.7
Version: 1.1.8
Author: Drip
Author URI: https://www.drip.com/
License: GPLv2
WC requires at least: 3.0
WC tested up to: 9.3.1
WC tested up to: 9.3.3
*/

defined( 'ABSPATH' ) || die( 'Executing outside of the WordPress context.' );
Expand Down
6 changes: 5 additions & 1 deletion readme.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ Contributors: getdrip
Tags: ecommerce, emailmarketing, marketingautomation, emailmarketingautomation, woocommerce, drip
Requires at least: 4.6
Tested up to: 6.6.2
Stable tag: 1.1.7
Stable tag: 1.1.8
Requires PHP: 5.6
License: GPLv2
License URI: https://www.gnu.org/licenses/gpl-2.0.html
Expand Down Expand Up @@ -74,6 +74,10 @@ The philosophy behind this plugin is to do as little as possible in it, and as m

* Your change here!

= 1.1.8 =

* Fix deprecation warnings for PHP 8.2

= 1.1.7 =

* Implement `drip_set_snippet_script_type` and `drip_set_snippet_script_additional_attributes` filters to add custom attributes in Drip's JS Snippet
Expand Down
4 changes: 2 additions & 2 deletions src/class-drip-woocommerce-plugin-view.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@ private function integrations_url() {
$account_id = WC_Admin_Settings::get_option( Drip_Woocommerce_Settings::ACCOUNT_ID_KEY );
$account_segment = '';
if ( ! empty( $account_id ) ) {
$account_segment = "${account_id}/";
$account_segment = "{$account_id}/";
}
return "<a href=\"https://www.getdrip.com/${account_segment}integrations/drip_woocommerce\">Integration Settings</a>";
return "<a href=\"https://www.getdrip.com/{$account_segment}integrations/drip_woocommerce\">Integration Settings</a>";
}
}
2 changes: 1 addition & 1 deletion src/snippet.js.php
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
var dc = document.createElement('script');
dc.type = '<?php echo esc_js( $script_type ); ?>';
dc.async = true;
dc.src = '//tag.getdrip.com/<?php echo esc_js( $account_id ); ?>.js';
dc.src = 'https://tag.getdrip.com/<?php echo esc_js( $account_id ); ?>.js';
<?php
if(is_array($script_additional_attributes)) {
foreach($script_additional_attributes as $attname => $attval) {
Expand Down

0 comments on commit 8b4a77e

Please sign in to comment.