Skip to content

Changed logic, redirect to cart #2

Changed logic, redirect to cart

Changed logic, redirect to cart #2

Triggered via pull request November 18, 2024 20:36
Status Failure
Total duration 23s
Artifacts

wpcs.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
VIPCS: includes/class-cart.php#L12
Use placeholders and $wpdb->prepare(); found interpolated variable $cart_id at "SELECT * FROM {$wpdb->prefix}extralsc_wsc_carts WHERE cart_id = $cart_id"
VIPCS: includes/class-product.php#L20
error_log() found. Debug code should not normally be used in production.
VIPCS: includes/class-cart-sharing.php#L10
Use placeholders and $wpdb->prepare(); found interpolated variable $sharing_id at "SELECT * FROM {$wpdb->prefix}extralsc_wsc_cart_sharing WHERE sharing_id = $sharing_id"
VIPCS: includes/class-cart-item.php#L13
Use placeholders and $wpdb->prepare(); found interpolated variable $cart_item_id at "SELECT * FROM {$wpdb->prefix}extralsc_wsc_cart_items WHERE cart_item_id = $cart_item_id"
VIPCS: includes/class-cart-item.php#L25
Use placeholders and $wpdb->prepare(); found interpolated variable $table_name at "SELECT * FROM $table_name WHERE cart_id = %d"
VIPCS: includes/class-cart-item.php#L26
Use placeholders and $wpdb->prepare(); found $query
VIPCS: includes/class-cart-session.php#L10
Use placeholders and $wpdb->prepare(); found interpolated variable $session_id at "SELECT * FROM {$wpdb->prefix}extralsc_wsc_cart_sessions WHERE session_id = $session_id"
VIPCS: extralsc-woocommerce-share-cart.php#L94
Use placeholders and $wpdb->prepare(); found $sql_carts
VIPCS: extralsc-woocommerce-share-cart.php#L98
Use placeholders and $wpdb->prepare(); found $sql_products
VIPCS: extralsc-woocommerce-share-cart.php#L102
Use placeholders and $wpdb->prepare(); found $sql_cart_items
VIPCS
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/