| @@ -8,15 +8,25 @@ | ||
| 8 | 8 | declare( strict_types=1 ); |
| 9 | 9 | |
| 10 | 10 | namespace Packetery\Module\Order; |
| 11 | 11 | |
| 12 | -use Packetery\Core\Helper; | |
| 12 | +use DateTimeImmutable; | |
| 13 | +use Exception; | |
| 14 | +use Packetery\Core\CoreHelper; | |
| 15 | +use Packetery\Core\Entity; | |
| 13 | 16 | use Packetery\Core\Entity\Order; |
| 14 | 17 | use Packetery\Core\Entity\PickupPoint; |
| 15 | -use Packetery\Core\Entity\Size; | |
| 16 | -use Packetery\Core\Entity\Address; | |
| 17 | 18 | use Packetery\Module\Carrier; |
| 18 | -use Packetery\Module\ShippingMethod; | |
| 19 | +use Packetery\Module\Carrier\PacketaPickupPointsConfig; | |
| 20 | +use Packetery\Module\CustomsDeclaration; | |
| 21 | +use Packetery\Module\Exception\DeleteErrorException; | |
| 22 | +use Packetery\Module\Exception\InvalidCarrierException; | |
| 23 | +use Packetery\Module\Framework\WcAdapter; | |
| 24 | +use Packetery\Module\ModuleHelper; | |
| 25 | +use Packetery\Module\Shipping\ShippingProvider; | |
| 26 | +use Packetery\Module\WcLogger; | |
| 27 | +use Packetery\Module\WpdbAdapter; | |
| 28 | +use WC_Order; | |
| 19 | 29 | use WP_Post; |
| 20 | 30 | |
| 21 | 31 | /** |
| 22 | 32 | * Class Repository. |
| @@ -25,13 +35,13 @@ | ||
| 25 | 35 | */ |
| 26 | 36 | class Repository { |
| 27 | 37 | |
| 28 | 38 | /** |
| 29 | - * Wpdb. | |
| 39 | + * WpdbAdapter. | |
| 30 | 40 | * |
| 31 | - * @var \wpdb | |
| 41 | + * @var WpdbAdapter | |
| 32 | 42 | */ |
| 33 | - private $wpdb; | |
| 43 | + private $wpdbAdapter; | |
| 34 | 44 | |
| 35 | 45 | /** |
| 36 | 46 | * Order factory. |
| 37 | 47 | * |
| @@ -39,28 +49,78 @@ | ||
| 39 | 49 | */ |
| 40 | 50 | private $builder; |
| 41 | 51 | |
| 42 | 52 | /** |
| 53 | + * CoreHelper. | |
| 54 | + * | |
| 55 | + * @var CoreHelper | |
| 56 | + */ | |
| 57 | + private $coreHelper; | |
| 58 | + | |
| 59 | + /** | |
| 60 | + * Internal pickup points config. | |
| 61 | + * | |
| 62 | + * @var PacketaPickupPointsConfig | |
| 63 | + */ | |
| 64 | + private $pickupPointsConfig; | |
| 65 | + | |
| 66 | + /** | |
| 67 | + * Carrier repository. | |
| 68 | + * | |
| 69 | + * @var Carrier\EntityRepository | |
| 70 | + */ | |
| 71 | + private $carrierRepository; | |
| 72 | + | |
| 73 | + /** | |
| 74 | + * Customs declaration repository. | |
| 75 | + * | |
| 76 | + * @var CustomsDeclaration\Repository | |
| 77 | + */ | |
| 78 | + private $customsDeclarationRepository; | |
| 79 | + | |
| 80 | + /** | |
| 81 | + * @var WcAdapter | |
| 82 | + */ | |
| 83 | + private $wcAdapter; | |
| 84 | + | |
| 85 | + /** | |
| 43 | 86 | * Repository constructor. |
| 44 | 87 | * |
| 45 | - * @param \wpdb $wpdb Wpdb. | |
| 46 | - * @param Builder $orderFactory Order factory. | |
| 88 | + * @param WpdbAdapter $wpdbAdapter WpdbAdapter. | |
| 89 | + * @param Builder $orderFactory Order factory. | |
| 90 | + * @param CoreHelper $coreHelper CoreHelper. | |
| 91 | + * @param PacketaPickupPointsConfig $pickupPointsConfig Internal pickup points config. | |
| 92 | + * @param Carrier\EntityRepository $carrierRepository Carrier repository. | |
| 93 | + * @param CustomsDeclaration\Repository $customsDeclarationRepository Customs declaration repository. | |
| 47 | 94 | */ |
| 48 | - public function __construct( \wpdb $wpdb, Builder $orderFactory ) { | |
| 49 | - $this->wpdb = $wpdb; | |
| 50 | - $this->builder = $orderFactory; | |
| 95 | + public function __construct( | |
| 96 | + WpdbAdapter $wpdbAdapter, | |
| 97 | + Builder $orderFactory, | |
| 98 | + CoreHelper $coreHelper, | |
| 99 | + PacketaPickupPointsConfig $pickupPointsConfig, | |
| 100 | + Carrier\EntityRepository $carrierRepository, | |
| 101 | + CustomsDeclaration\Repository $customsDeclarationRepository, | |
| 102 | + WcAdapter $wcAdapter | |
| 103 | + ) { | |
| 104 | + $this->wpdbAdapter = $wpdbAdapter; | |
| 105 | + $this->builder = $orderFactory; | |
| 106 | + $this->coreHelper = $coreHelper; | |
| 107 | + $this->pickupPointsConfig = $pickupPointsConfig; | |
| 108 | + $this->carrierRepository = $carrierRepository; | |
| 109 | + $this->customsDeclarationRepository = $customsDeclarationRepository; | |
| 110 | + $this->wcAdapter = $wcAdapter; | |
| 51 | 111 | } |
| 52 | 112 | |
| 53 | 113 | /** |
| 54 | 114 | * Applies custom order status filter. |
| 55 | 115 | * |
| 56 | - * @param array $clauses Query clauses. | |
| 57 | - * @param \WP_Query $queryObject WP Query. | |
| 58 | - * @param array $paramValues Param values. | |
| 116 | + * @param array<string, string|null> $clauses Query clauses. | |
| 117 | + * @param \WP_Query|null $queryObject WP Query. | |
| 118 | + * @param array<string, string|null> $paramValues Param values. | |
| 59 | 119 | * |
| 60 | 120 | * @return void |
| 61 | 121 | */ |
| 62 | - private function applyCustomFilters( array &$clauses, \WP_Query $queryObject, array $paramValues ): void { | |
| 122 | + private function applyCustomFilters( array &$clauses, ?\WP_Query $queryObject, array $paramValues ): void { | |
| 63 | 123 | /** |
| 64 | 124 | * Filters order statuses to exclude in Packeta order list filtering. |
| 65 | 125 | * |
| 66 | 126 | * @since 1.4 |
| @@ -65,72 +125,71 @@ | ||
| 65 | 125 | * |
| 66 | 126 | * @since 1.4 |
| 67 | 127 | * |
| 68 | 128 | * @param array $orderStatusesToExclude Order statuses to exclude. |
| 69 | - * @param \WP_Query $queryObject WP Query. | |
| 129 | + * @param \WP_Query|null $queryObject WP Query. | |
| 70 | 130 | * @param array $paramValues Param values. |
| 71 | 131 | */ |
| 72 | - $orderStatusesToExclude = apply_filters( 'packetery_exclude_orders_with_status', [], $queryObject, $paramValues ); | |
| 73 | - if ( ! $orderStatusesToExclude ) { | |
| 132 | + $orderStatusesToExclude = (array) apply_filters( 'packetery_exclude_orders_with_status', [], $queryObject, $paramValues ); | |
| 133 | + if ( count( $orderStatusesToExclude ) === 0 ) { | |
| 74 | 134 | return; |
| 75 | 135 | } |
| 76 | 136 | |
| 77 | - $sqlStatusesToExclude = implode( | |
| 78 | - ',', | |
| 79 | - array_map( | |
| 80 | - function ( string $orderStatus ): string { | |
| 81 | - return sprintf( '"%s"', $this->wpdb->_real_escape( $orderStatus ) ); | |
| 82 | - }, | |
| 83 | - $orderStatusesToExclude | |
| 84 | - ) | |
| 85 | - ); | |
| 86 | - | |
| 87 | - $clauses['where'] .= sprintf( ' AND `%s`.`post_status` NOT IN (%s)', $this->wpdb->posts, $sqlStatusesToExclude ); | |
| 137 | + if ( ModuleHelper::isHposEnabled() ) { | |
| 138 | + $clauses['where'] .= sprintf( ' AND `%s`.`status` NOT IN (%s)', $this->wpdbAdapter->wcOrders, $this->wpdbAdapter->prepareInClause( $orderStatusesToExclude ) ); | |
| 139 | + } else { | |
| 140 | + $clauses['where'] .= sprintf( ' AND `%s`.`post_status` NOT IN (%s)', $this->wpdbAdapter->posts, $this->wpdbAdapter->prepareInClause( $orderStatusesToExclude ) ); | |
| 141 | + } | |
| 88 | 142 | } |
| 89 | 143 | |
| 90 | 144 | /** |
| 91 | - * Extends WP_Query to include custom table. | |
| 145 | + * Extends SQL query clauses. | |
| 92 | 146 | * |
| 93 | - * @link https://wordpress.stackexchange.com/questions/50305/how-to-extend-wp-query-to-include-custom-table-in-query | |
| 147 | + * @param array<string, string> $clauses Clauses. | |
| 148 | + * @param \WP_Query|null $queryObject Query object. | |
| 149 | + * @param array<string, string|null> $paramValues Param values. | |
| 94 | 150 | * |
| 95 | - * @param array $clauses Clauses. | |
| 96 | - * @param \WP_Query $queryObject WP_Query. | |
| 97 | - * @param array $paramValues Param values. | |
| 98 | - * | |
| 99 | - * @return array | |
| 151 | + * @return array<string, string> | |
| 100 | 152 | */ |
| 101 | - public function processPostClauses( array $clauses, \WP_Query $queryObject, array $paramValues ): array { | |
| 102 | - if ( isset( $queryObject->query['post_type'] ) && | |
| 103 | - ( | |
| 104 | - 'shop_order' === $queryObject->query['post_type'] || | |
| 105 | - ( is_array( $queryObject->query['post_type'] ) && in_array( 'shop_order', $queryObject->query['post_type'], true ) ) | |
| 106 | - ) | |
| 107 | - ) { | |
| 108 | - // TODO: Introduce variable. | |
| 109 | - $clauses['join'] .= ' LEFT JOIN `' . $this->wpdb->packetery_order . '` ON `' . $this->wpdb->packetery_order . '`.`id` = `' . $this->wpdb->posts . '`.`id`'; | |
| 153 | + public function processClauses( array $clauses, ?\WP_Query $queryObject, array $paramValues ): array { | |
| 154 | + if ( ModuleHelper::isHposEnabled() ) { | |
| 155 | + $clauses['join'] .= ' LEFT JOIN `' . $this->wpdbAdapter->packeteryOrder . '` ON `' . $this->wpdbAdapter->packeteryOrder . '`.`id` = `' . $this->wpdbAdapter->wcOrders . '`.`id`'; | |
| 156 | + } else { | |
| 157 | + $clauses['join'] .= ' LEFT JOIN `' . $this->wpdbAdapter->packeteryOrder . '` ON `' . $this->wpdbAdapter->packeteryOrder . '`.`id` = `' . $this->wpdbAdapter->posts . '`.`id`'; | |
| 158 | + } | |
| 110 | 159 | |
| 111 | - if ( $paramValues['packetery_carrier_id'] ) { | |
| 112 | - $clauses['where'] .= ' AND `' . $this->wpdb->packetery_order . '`.`carrier_id` = "' . $this->wpdb->_real_escape( $paramValues['packetery_carrier_id'] ) . '"'; | |
| 113 | - $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 160 | + if ( isset( $paramValues['packetery_carrier_id'] ) && $paramValues['packetery_carrier_id'] !== '' ) { | |
| 161 | + $clauses['where'] .= ' AND `' . $this->wpdbAdapter->packeteryOrder . '`.`carrier_id` = "' . esc_sql( $paramValues['packetery_carrier_id'] ) . '"'; | |
| 162 | + $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 163 | + } | |
| 164 | + if ( isset( $paramValues['packetery_to_submit'] ) && $paramValues['packetery_to_submit'] !== '' ) { | |
| 165 | + $clauses['where'] .= ' AND `' . $this->wpdbAdapter->packeteryOrder . '`.`carrier_id` IS NOT NULL '; | |
| 166 | + $clauses['where'] .= ' AND `' . $this->wpdbAdapter->packeteryOrder . '`.`is_exported` = false '; | |
| 167 | + $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 168 | + } | |
| 169 | + if ( isset( $paramValues['packetery_to_print'] ) && $paramValues['packetery_to_print'] !== '' ) { | |
| 170 | + $clauses['where'] .= ' AND `' . $this->wpdbAdapter->packeteryOrder . '`.`packet_id` IS NOT NULL '; | |
| 171 | + $clauses['where'] .= ' AND `' . $this->wpdbAdapter->packeteryOrder . '`.`is_label_printed` = false '; | |
| 172 | + $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 173 | + } | |
| 174 | + if ( isset( $paramValues['packetery_order_type'] ) && $paramValues['packetery_order_type'] !== '' ) { | |
| 175 | + if ( $paramValues['packetery_order_type'] === Entity\Carrier::INTERNAL_PICKUP_POINTS_ID ) { | |
| 176 | + $comparison = 'IN'; | |
| 177 | + } else { | |
| 178 | + $comparison = 'NOT IN'; | |
| 114 | 179 | } |
| 115 | - if ( $paramValues['packetery_to_submit'] ) { | |
| 116 | - $clauses['where'] .= ' AND `' . $this->wpdb->packetery_order . '`.`carrier_id` IS NOT NULL '; | |
| 117 | - $clauses['where'] .= ' AND `' . $this->wpdb->packetery_order . '`.`is_exported` = false '; | |
| 118 | - $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 180 | + $internalCarriers = array_keys( $this->carrierRepository->getNonFeedCarriers() ); | |
| 181 | + $internalCarriers[] = Entity\Carrier::INTERNAL_PICKUP_POINTS_ID; | |
| 182 | + $clauses['where'] .= ' AND `' . $this->wpdbAdapter->packeteryOrder . '`.`carrier_id` ' . $comparison . ' (' . $this->wpdbAdapter->prepareInClause( $internalCarriers ) . ')'; | |
| 183 | + $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 184 | + } | |
| 185 | + if ( isset( $paramValues['orderby'] ) && $paramValues['orderby'] === 'packetery_packet_stored_until' ) { | |
| 186 | + if ( $paramValues['order'] === 'asc' ) { | |
| 187 | + $clauses['orderby'] = '`' . $this->wpdbAdapter->packeteryOrder . '`.`stored_until` ASC'; | |
| 119 | 188 | } |
| 120 | - if ( $paramValues['packetery_to_print'] ) { | |
| 121 | - $clauses['where'] .= ' AND `' . $this->wpdb->packetery_order . '`.`packet_id` IS NOT NULL '; | |
| 122 | - $clauses['where'] .= ' AND `' . $this->wpdb->packetery_order . '`.`is_label_printed` = false '; | |
| 123 | - $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 189 | + if ( $paramValues['order'] === 'desc' ) { | |
| 190 | + $clauses['orderby'] = '`' . $this->wpdbAdapter->packeteryOrder . '`.`stored_until` DESC'; | |
| 124 | 191 | } |
| 125 | - if ( $paramValues['packetery_order_type'] ) { | |
| 126 | - if ( Carrier\Repository::INTERNAL_PICKUP_POINTS_ID === $paramValues['packetery_order_type'] ) { | |
| 127 | - $clauses['where'] .= ' AND `' . $this->wpdb->packetery_order . '`.`carrier_id` = "' . $this->wpdb->_real_escape( Carrier\Repository::INTERNAL_PICKUP_POINTS_ID ) . '"'; | |
| 128 | - } else { | |
| 129 | - $clauses['where'] .= ' AND `' . $this->wpdb->packetery_order . '`.`carrier_id` != "' . $this->wpdb->_real_escape( Carrier\Repository::INTERNAL_PICKUP_POINTS_ID ) . '"'; | |
| 130 | - } | |
| 131 | - $this->applyCustomFilters( $clauses, $queryObject, $paramValues ); | |
| 132 | - } | |
| 133 | 192 | } |
| 134 | 193 | |
| 135 | 194 | return $clauses; |
| 136 | 195 | } |
| @@ -139,184 +198,154 @@ | ||
| 139 | 198 | * Create table to store orders. |
| 140 | 199 | * |
| 141 | 200 | * @return bool |
| 142 | 201 | */ |
| 143 | - public function createTable(): bool { | |
| 144 | - $wpdb = $this->wpdb; | |
| 202 | + public function createOrAlterTable(): bool { | |
| 203 | + $createTableQuery = 'CREATE TABLE ' . $this->wpdbAdapter->packeteryOrder . ' ( | |
| 204 | + `id` bigint(20) unsigned NOT NULL, | |
| 205 | + `carrier_id` varchar(15) NOT NULL, | |
| 206 | + `is_exported` tinyint(1) NOT NULL, | |
| 207 | + `packet_id` varchar(15) NULL, | |
| 208 | + `packet_claim_id` varchar(15) NULL, | |
| 209 | + `packet_claim_password` varchar(10) NULL, | |
| 210 | + `consign_password` varchar(10) NULL, | |
| 211 | + `is_label_printed` tinyint(1) NOT NULL, | |
| 212 | + `point_id` varchar(50) NULL, | |
| 213 | + `point_place` varchar(100) NULL, | |
| 214 | + `point_name` varchar(150) NULL, | |
| 215 | + `point_url` varchar(255) NULL, | |
| 216 | + `point_street` varchar(120) NULL, | |
| 217 | + `point_zip` varchar(10) NULL, | |
| 218 | + `point_city` varchar(70) NULL, | |
| 219 | + `address_validated` tinyint(1) NOT NULL, | |
| 220 | + `delivery_address` text NULL, | |
| 221 | + `weight` float NULL, | |
| 222 | + `length` float NULL, | |
| 223 | + `width` float NULL, | |
| 224 | + `height` float NULL, | |
| 225 | + `adult_content` tinyint(1) NULL, | |
| 226 | + `value` double NULL, | |
| 227 | + `cod` double NULL, | |
| 228 | + `api_error_message` text NULL, | |
| 229 | + `api_error_date` datetime NULL, | |
| 230 | + `carrier_number` varchar(50) NULL, | |
| 231 | + `packet_status` varchar(30) NULL, | |
| 232 | + `stored_until` date NULL, | |
| 233 | + `deliver_on` date NULL, | |
| 234 | + `car_delivery_id` varchar(15) NULL, | |
| 235 | + PRIMARY KEY (`id`) | |
| 236 | + ) ' . $this->wpdbAdapter->get_charset_collate(); | |
| 145 | 237 | |
| 146 | - return $wpdb->query( | |
| 147 | - 'CREATE TABLE IF NOT EXISTS `' . $wpdb->packetery_order . '` ( | |
| 148 | - `id` bigint(20) unsigned NOT NULL, | |
| 149 | - `carrier_id` varchar(255) NOT NULL, | |
| 150 | - `is_exported` boolean NOT NULL, | |
| 151 | - `packet_id` varchar(255) NULL, | |
| 152 | - `is_label_printed` boolean NOT NULL, | |
| 153 | - `point_id` varchar(255) NULL, | |
| 154 | - `point_name` varchar(255) NULL, | |
| 155 | - `point_url` varchar(255) NULL, | |
| 156 | - `point_street` varchar(255) NULL, | |
| 157 | - `point_zip` varchar(255) NULL, | |
| 158 | - `point_city` varchar(255) NULL, | |
| 159 | - `address_validated` boolean NOT NULL, | |
| 160 | - `delivery_address` TEXT NULL, | |
| 161 | - `weight` float NULL, | |
| 162 | - `length` float NULL, | |
| 163 | - `width` float NULL, | |
| 164 | - `height` float NULL, | |
| 165 | - `adult_content` boolean NULL, | |
| 166 | - `value` double NULL, | |
| 167 | - `cod` double NULL, | |
| 168 | - `carrier_number` varchar(255) NULL, | |
| 169 | - `packet_status` varchar(255) NULL, | |
| 170 | - PRIMARY KEY (`id`) | |
| 171 | - ) ' . $wpdb->get_charset_collate() | |
| 172 | - ); | |
| 238 | + return $this->wpdbAdapter->dbDelta( $createTableQuery, $this->wpdbAdapter->packeteryOrder ); | |
| 173 | 239 | } |
| 174 | 240 | |
| 175 | 241 | /** |
| 176 | - * Drop table used to store orders. | |
| 242 | + * @throws InvalidCarrierException InvalidCarrierException. | |
| 177 | 243 | */ |
| 178 | - public function drop(): void { | |
| 179 | - $wpdb = $this->wpdb; | |
| 180 | - $wpdb->query( 'DROP TABLE IF EXISTS `' . $wpdb->packetery_order . '`' ); | |
| 244 | + public function getById( int $id ): ?Order { | |
| 245 | + $wcOrder = $this->getWcOrderById( $id ); | |
| 246 | + if ( $wcOrder === null ) { | |
| 247 | + return null; | |
| 248 | + } | |
| 249 | + | |
| 250 | + return $this->getByWcOrder( $wcOrder ); | |
| 181 | 251 | } |
| 182 | 252 | |
| 183 | 253 | /** |
| 184 | - * Gets order data. | |
| 185 | - * | |
| 186 | - * @param int $id Order id. | |
| 187 | - * | |
| 188 | - * @return Order|null | |
| 254 | + * Ignores InvalidCarrierException. | |
| 189 | 255 | */ |
| 190 | - public function getById( int $id ): ?Order { | |
| 191 | - $wcOrder = wc_get_order( $id ); | |
| 192 | - if ( ! $wcOrder instanceof \WC_Order ) { | |
| 256 | + public function getByIdWithValidCarrier( int $id ): ?Order { | |
| 257 | + $wcOrder = $this->getWcOrderById( $id ); | |
| 258 | + if ( $wcOrder === null ) { | |
| 193 | 259 | return null; |
| 194 | 260 | } |
| 195 | 261 | |
| 196 | - return $this->getByWcOrder( $wcOrder ); | |
| 262 | + return $this->getByWcOrderWithValidCarrier( $wcOrder ); | |
| 197 | 263 | } |
| 198 | 264 | |
| 199 | 265 | /** |
| 200 | - * Gets order by wc order. | |
| 266 | + * Returns Packeta order, or null. | |
| 201 | 267 | * |
| 202 | - * @param \WC_Order $wcOrder WC Order. | |
| 268 | + * @param int $id Order ID. | |
| 203 | 269 | * |
| 204 | 270 | * @return Order|null |
| 205 | 271 | */ |
| 206 | - public function getByWcOrder( \WC_Order $wcOrder ): ?Order { | |
| 207 | - if ( ! $wcOrder->has_shipping_method( ShippingMethod::PACKETERY_METHOD_ID ) ) { | |
| 272 | + public function findById( int $id ): ?Order { | |
| 273 | + $wcOrder = $this->getWcOrderById( $id ); | |
| 274 | + if ( $wcOrder === null ) { | |
| 208 | 275 | return null; |
| 209 | 276 | } |
| 210 | 277 | |
| 211 | - $wpdb = $this->wpdb; | |
| 278 | + $result = $this->getDataById( $wcOrder->get_id() ); | |
| 279 | + if ( $result === null ) { | |
| 280 | + return null; | |
| 281 | + } | |
| 212 | 282 | |
| 213 | - $result = $wpdb->get_row( | |
| 214 | - $wpdb->prepare( | |
| 215 | - ' | |
| 216 | - SELECT o.* FROM `' . $wpdb->packetery_order . '` o | |
| 217 | - JOIN `' . $wpdb->posts . '` as wp_p ON wp_p.ID = o.id | |
| 218 | - WHERE o.`id` = %d', | |
| 219 | - $wcOrder->get_id() | |
| 220 | - ) | |
| 221 | - ); | |
| 222 | - | |
| 223 | - if ( ! $result ) { | |
| 283 | + try { | |
| 284 | + return $this->builder->build( $wcOrder, $result ); | |
| 285 | + } catch ( InvalidCarrierException $invalidCarrierException ) { | |
| 224 | 286 | return null; |
| 225 | 287 | } |
| 226 | - | |
| 227 | - $partialOrder = $this->createPartialOrder( $result ); | |
| 228 | - return $this->builder->finalize( $wcOrder, $partialOrder ); | |
| 229 | 288 | } |
| 230 | 289 | |
| 231 | 290 | /** |
| 232 | - * Creates partial order. | |
| 291 | + * Gets Packeta order data by id. | |
| 233 | 292 | * |
| 234 | - * @param \stdClass $result DB result. | |
| 293 | + * @param int $id Order id. | |
| 235 | 294 | * |
| 236 | - * @return Order | |
| 295 | + * @return object|null | |
| 237 | 296 | */ |
| 238 | - private function createPartialOrder( \stdClass $result ): Order { | |
| 239 | - $partialOrder = new Order( | |
| 240 | - $result->id, | |
| 241 | - $result->carrier_id | |
| 297 | + public function getDataById( int $id ): ?object { | |
| 298 | + return $this->wpdbAdapter->get_row( | |
| 299 | + $this->wpdbAdapter->prepare( | |
| 300 | + ' | |
| 301 | + SELECT `o`.* FROM `' . $this->wpdbAdapter->packeteryOrder . '` `o` | |
| 302 | + ' . $this->getWcOrderJoinClause() . ' | |
| 303 | + WHERE `o`.`id` = %d', | |
| 304 | + $id | |
| 305 | + ) | |
| 242 | 306 | ); |
| 307 | + } | |
| 243 | 308 | |
| 244 | - $orderWeight = $this->parseFloat( $result->weight ); | |
| 245 | - if ( null !== $orderWeight ) { | |
| 246 | - $partialOrder->setWeight( $orderWeight ); | |
| 309 | + private function getDataByWcOrder( WC_Order $wcOrder ): ?object { | |
| 310 | + if ( ! ShippingProvider::wcOrderHasOurMethod( $wcOrder ) ) { | |
| 311 | + return null; | |
| 247 | 312 | } |
| 248 | - $partialOrder->setPacketId( $result->packet_id ); | |
| 249 | - $partialOrder->setSize( new Size( $this->parseFloat( $result->length ), $this->parseFloat( $result->width ), $this->parseFloat( $result->height ) ) ); | |
| 250 | - $partialOrder->setIsExported( (bool) $result->is_exported ); | |
| 251 | - $partialOrder->setIsLabelPrinted( (bool) $result->is_label_printed ); | |
| 252 | - $partialOrder->setCarrierNumber( $result->carrier_number ); | |
| 253 | - $partialOrder->setPacketStatus( $result->packet_status ); | |
| 254 | - $partialOrder->setAddressValidated( (bool) $result->address_validated ); | |
| 255 | - $partialOrder->setAdultContent( $this->parseBool( $result->adult_content ) ); | |
| 256 | - $partialOrder->setValue( $this->parseFloat( $result->value ) ); | |
| 257 | - $partialOrder->setCod( $this->parseFloat( $result->cod ) ); | |
| 258 | 313 | |
| 259 | - if ( $result->delivery_address ) { | |
| 260 | - $deliveryAddressDecoded = json_decode( $result->delivery_address, false ); | |
| 261 | - $deliveryAddress = new Address( | |
| 262 | - $deliveryAddressDecoded->street, | |
| 263 | - $deliveryAddressDecoded->city, | |
| 264 | - $deliveryAddressDecoded->zip | |
| 265 | - ); | |
| 266 | - | |
| 267 | - $deliveryAddress->setHouseNumber( $deliveryAddressDecoded->houseNumber ); | |
| 268 | - $deliveryAddress->setLongitude( $deliveryAddressDecoded->longitude ); | |
| 269 | - $deliveryAddress->setLatitude( $deliveryAddressDecoded->latitude ); | |
| 270 | - $deliveryAddress->setCounty( $deliveryAddressDecoded->county ); | |
| 271 | - | |
| 272 | - $partialOrder->setDeliveryAddress( $deliveryAddress ); | |
| 314 | + $packeteryOrderData = $this->getDataById( $wcOrder->get_id() ); | |
| 315 | + if ( ! is_object( $packeteryOrderData ) ) { | |
| 316 | + return null; | |
| 273 | 317 | } |
| 274 | 318 | |
| 275 | - if ( null !== $result->point_id ) { | |
| 276 | - $pickUpPoint = new PickupPoint( | |
| 277 | - $result->point_id, | |
| 278 | - $result->point_name, | |
| 279 | - $result->point_city, | |
| 280 | - $result->point_zip, | |
| 281 | - $result->point_street, | |
| 282 | - $result->point_url | |
| 283 | - ); | |
| 284 | - | |
| 285 | - $partialOrder->setPickupPoint( $pickUpPoint ); | |
| 286 | - } | |
| 287 | - | |
| 288 | - return $partialOrder; | |
| 319 | + return $packeteryOrderData; | |
| 289 | 320 | } |
| 290 | 321 | |
| 291 | 322 | /** |
| 292 | - * Parses string value as float. | |
| 293 | - * | |
| 294 | - * @param string|float|null $value Value. | |
| 295 | - * | |
| 296 | - * @return float|null | |
| 323 | + * @throws InvalidCarrierException | |
| 297 | 324 | */ |
| 298 | - private function parseFloat( $value ): ?float { | |
| 299 | - if ( null === $value || '' === $value ) { | |
| 325 | + public function getByWcOrder( WC_Order $wcOrder ): ?Order { | |
| 326 | + $packeteryOrderData = $this->getDataByWcOrder( $wcOrder ); | |
| 327 | + if ( $packeteryOrderData === null ) { | |
| 300 | 328 | return null; |
| 301 | 329 | } |
| 302 | 330 | |
| 303 | - return (float) $value; | |
| 331 | + return $this->builder->build( $wcOrder, $packeteryOrderData ); | |
| 304 | 332 | } |
| 305 | 333 | |
| 306 | 334 | /** |
| 307 | - * Parses string value as float. | |
| 308 | - * | |
| 309 | - * @param string|int|null $value Value. | |
| 310 | - * | |
| 311 | - * @return bool|null | |
| 335 | + * Ignores InvalidCarrierException. | |
| 312 | 336 | */ |
| 313 | - private function parseBool( $value ): ?bool { | |
| 314 | - if ( null === $value || '' === $value ) { | |
| 337 | + public function getByWcOrderWithValidCarrier( WC_Order $wcOrder ): ?Order { | |
| 338 | + $packeteryOrderData = $this->getDataByWcOrder( $wcOrder ); | |
| 339 | + if ( $packeteryOrderData === null ) { | |
| 315 | 340 | return null; |
| 316 | 341 | } |
| 317 | 342 | |
| 318 | - return (bool) $value; | |
| 343 | + try { | |
| 344 | + return $this->builder->build( $wcOrder, $packeteryOrderData ); | |
| 345 | + } catch ( InvalidCarrierException $invalidCarrierException ) { | |
| 346 | + return null; | |
| 347 | + } | |
| 319 | 348 | } |
| 320 | 349 | |
| 321 | 350 | /** |
| 322 | 351 | * Transforms order to DB array. |
| @@ -322,44 +351,58 @@ | ||
| 322 | 351 | * Transforms order to DB array. |
| 323 | 352 | * |
| 324 | 353 | * @param Order $order Order. |
| 325 | 354 | * |
| 326 | - * @return array | |
| 355 | + * @return array<string, bool|float|int|string|null> | |
| 327 | 356 | */ |
| 328 | 357 | private function orderToDbArray( Order $order ): array { |
| 329 | 358 | $point = $order->getPickupPoint(); |
| 330 | - if ( null === $point ) { | |
| 359 | + if ( $point === null ) { | |
| 331 | 360 | $point = new PickupPoint(); |
| 332 | 361 | } |
| 333 | 362 | |
| 334 | 363 | $deliveryAddress = null; |
| 335 | - if ( $order->isAddressValidated() && $order->getDeliveryAddress() ) { | |
| 364 | + if ( $order->isAddressValidated() && $order->getDeliveryAddress() !== null ) { | |
| 336 | 365 | $deliveryAddress = wp_json_encode( $order->getDeliveryAddress()->export() ); |
| 337 | 366 | } |
| 338 | 367 | |
| 368 | + $apiErrorDateTime = $order->getLastApiErrorDateTime(); | |
| 369 | + if ( $apiErrorDateTime !== null ) { | |
| 370 | + $apiErrorDateTime = $apiErrorDateTime->format( CoreHelper::MYSQL_DATETIME_FORMAT ); | |
| 371 | + } | |
| 372 | + | |
| 339 | 373 | $data = [ |
| 340 | - 'id' => (int) $order->getNumber(), | |
| 341 | - 'carrier_id' => $order->getCarrierId(), | |
| 342 | - 'is_exported' => (int) $order->isExported(), | |
| 343 | - 'packet_id' => $order->getPacketId(), | |
| 344 | - 'packet_status' => $order->getPacketStatus(), | |
| 345 | - 'is_label_printed' => (int) $order->isLabelPrinted(), | |
| 346 | - 'carrier_number' => $order->getCarrierNumber(), | |
| 347 | - 'weight' => $order->getWeight(), | |
| 348 | - 'point_id' => $point->getId(), | |
| 349 | - 'point_name' => $point->getName(), | |
| 350 | - 'point_url' => $point->getUrl(), | |
| 351 | - 'point_street' => $point->getStreet(), | |
| 352 | - 'point_zip' => $point->getZip(), | |
| 353 | - 'point_city' => $point->getCity(), | |
| 354 | - 'address_validated' => (int) $order->isAddressValidated(), | |
| 355 | - 'delivery_address' => $deliveryAddress, | |
| 356 | - 'length' => $order->getLength(), | |
| 357 | - 'width' => $order->getWidth(), | |
| 358 | - 'height' => $order->getHeight(), | |
| 359 | - 'adult_content' => $order->containsAdultContent(), | |
| 360 | - 'cod' => $order->getCod(), | |
| 361 | - 'value' => $order->getValue(), | |
| 374 | + 'id' => (int) $order->getNumber(), | |
| 375 | + 'carrier_id' => $order->getCarrier()->getId(), | |
| 376 | + 'is_exported' => (int) $order->isExported(), | |
| 377 | + 'packet_id' => $order->getPacketId(), | |
| 378 | + 'packet_claim_id' => $order->getPacketClaimId(), | |
| 379 | + 'packet_claim_password' => $order->getPacketClaimPassword(), | |
| 380 | + 'consign_password' => $order->getConsignPassword(), | |
| 381 | + 'packet_status' => $order->getPacketStatus(), | |
| 382 | + 'stored_until' => $this->coreHelper->getStringFromDateTime( $order->getStoredUntil(), CoreHelper::DATEPICKER_FORMAT ), | |
| 383 | + 'is_label_printed' => (int) $order->isLabelPrinted(), | |
| 384 | + 'carrier_number' => $order->getCarrierNumber(), | |
| 385 | + 'weight' => $order->getWeight(), | |
| 386 | + 'car_delivery_id' => $order->getCarDeliveryId(), | |
| 387 | + 'point_id' => $point->getId(), | |
| 388 | + 'point_place' => $point->getPlace(), | |
| 389 | + 'point_name' => $point->getName(), | |
| 390 | + 'point_url' => $point->getUrl(), | |
| 391 | + 'point_street' => $point->getStreet(), | |
| 392 | + 'point_zip' => $point->getZip(), | |
| 393 | + 'point_city' => $point->getCity(), | |
| 394 | + 'address_validated' => (int) $order->isAddressValidated(), | |
| 395 | + 'delivery_address' => $deliveryAddress, | |
| 396 | + 'length' => $order->getLength(), | |
| 397 | + 'width' => $order->getWidth(), | |
| 398 | + 'height' => $order->getHeight(), | |
| 399 | + 'adult_content' => $order->containsAdultContent(), | |
| 400 | + 'cod' => $order->getManualCod(), | |
| 401 | + 'value' => $order->getManualValue(), | |
| 402 | + 'api_error_message' => $order->getLastApiErrorMessage(), | |
| 403 | + 'api_error_date' => $apiErrorDateTime, | |
| 404 | + 'deliver_on' => $this->coreHelper->getStringFromDateTime( $order->getDeliverOn(), CoreHelper::DATEPICKER_FORMAT ), | |
| 362 | 405 | ]; |
| 363 | 406 | |
| 364 | 407 | return $data; |
| 365 | 408 | } |
| @@ -368,40 +411,105 @@ | ||
| 368 | 411 | * Saves order. |
| 369 | 412 | * |
| 370 | 413 | * @param Order $order Order. |
| 371 | 414 | * |
| 415 | + * @return int|false The number of rows updated, or false on error. | |
| 416 | + */ | |
| 417 | + public function save( Order $order ) { | |
| 418 | + return $this->saveData( $this->orderToDbArray( $order ) ); | |
| 419 | + } | |
| 420 | + | |
| 421 | + /** | |
| 422 | + * Saves order data. | |
| 423 | + * | |
| 424 | + * @param array<string, int|string|null|DateTimeImmutable> $orderData Order data. | |
| 425 | + * | |
| 426 | + * @return int|false The number of rows updated, or false on error. | |
| 427 | + */ | |
| 428 | + public function saveData( array $orderData ) { | |
| 429 | + $this->onBeforeDataInsertion( $orderData ); | |
| 430 | + | |
| 431 | + return $this->wpdbAdapter->insertReplaceHelper( $this->wpdbAdapter->packeteryOrder, $orderData, null, 'REPLACE' ); | |
| 432 | + } | |
| 433 | + | |
| 434 | + /** | |
| 435 | + * Calls logic before order data replace/insert. | |
| 436 | + * | |
| 437 | + * @param array<string, string|null|DateTimeImmutable> $orderData Order data. | |
| 372 | 438 | * @return void |
| 373 | 439 | */ |
| 374 | - public function save( Order $order ): void { | |
| 375 | - $this->wpdb->_insert_replace_helper( $this->wpdb->packetery_order, $this->orderToDbArray( $order ), null, 'REPLACE' ); | |
| 440 | + private function onBeforeDataInsertion( array $orderData ): void { | |
| 441 | + $pointId = $orderData['point_id'] ?? null; | |
| 442 | + $carrierId = $orderData['carrier_id'] ?? null; | |
| 443 | + /** | |
| 444 | + * Tells if Packeta debug logs are enabled. | |
| 445 | + * | |
| 446 | + * @since 1.7.2 | |
| 447 | + */ | |
| 448 | + $isLoggingActive = (bool) apply_filters( 'packeta_enable_debug_logs', false ); | |
| 449 | + | |
| 450 | + if ( ( $pointId !== null && $pointId !== '' ) || ! $isLoggingActive || ! $this->pickupPointsConfig->isInternalPickupPointCarrier( $carrierId ) ) { | |
| 451 | + return; | |
| 452 | + } | |
| 453 | + | |
| 454 | + $wcLogger = $this->wcAdapter->getLogger(); | |
| 455 | + $dataToLog = [ | |
| 456 | + 'order' => $orderData, | |
| 457 | + 'trace' => array_map( | |
| 458 | + static function ( array $item ): array { | |
| 459 | + return [ | |
| 460 | + 'file' => sprintf( '%s(%s)', $item['file'], $item['line'] ), | |
| 461 | + 'method' => sprintf( '%s%s%s(...)', $item['class'] ?? '', $item['type'] ?? '', $item['function'] ), | |
| 462 | + ]; | |
| 463 | + }, | |
| 464 | + ( new Exception() )->getTrace() | |
| 465 | + ), | |
| 466 | + ]; | |
| 467 | + $wcLogger->warning( | |
| 468 | + sprintf( | |
| 469 | + 'Required value for the pickup point order is not set: %s', | |
| 470 | + wp_json_encode( $dataToLog, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ) | |
| 471 | + ), | |
| 472 | + [ 'source' => 'packeta' ] | |
| 473 | + ); | |
| 376 | 474 | } |
| 377 | 475 | |
| 378 | 476 | /** |
| 379 | 477 | * Loads order entities by list of ids. |
| 380 | 478 | * |
| 381 | - * @param array $orderIds Order ids. | |
| 479 | + * @param string[] $orderIds Order ids. | |
| 382 | 480 | * |
| 383 | 481 | * @return Order[] |
| 384 | 482 | */ |
| 385 | 483 | public function getByIds( array $orderIds ): array { |
| 484 | + if ( count( $orderIds ) === 0 ) { | |
| 485 | + return []; | |
| 486 | + } | |
| 487 | + | |
| 488 | + $wpdbAdapter = $this->wpdbAdapter; | |
| 489 | + $ordersIdsPlaceholder = implode( ', ', array_fill( 0, count( $orderIds ), '%d' ) ); | |
| 490 | + $packeteryOrdersResult = $wpdbAdapter->get_results( | |
| 491 | + $wpdbAdapter->prepare( | |
| 492 | + 'SELECT * FROM `' . $wpdbAdapter->packeteryOrder . '` | |
| 493 | + WHERE `id` IN (' . $ordersIdsPlaceholder . ')', | |
| 494 | + $orderIds | |
| 495 | + ), | |
| 496 | + OBJECT_K | |
| 497 | + ); | |
| 498 | + | |
| 386 | 499 | $orderEntities = []; |
| 387 | - $posts = get_posts( | |
| 388 | - [ | |
| 389 | - 'post_type' => 'shop_order', | |
| 390 | - 'post__in' => $orderIds, | |
| 391 | - 'post_status' => [ 'any', 'trash' ], | |
| 392 | - 'nopaging' => true, | |
| 393 | - ] | |
| 394 | - ); | |
| 395 | - foreach ( $posts as $post ) { | |
| 396 | - $wcOrder = wc_get_order( $post ); | |
| 397 | - // In case WC_Order does not exist, result set is limited to existing records. | |
| 398 | - if ( ! $wcOrder ) { | |
| 500 | + foreach ( $orderIds as $orderId ) { | |
| 501 | + if ( ! isset( $packeteryOrdersResult[ $orderId ] ) ) { | |
| 399 | 502 | continue; |
| 400 | 503 | } |
| 401 | - $order = $this->getByWcOrder( $wcOrder ); | |
| 402 | - if ( $order ) { | |
| 403 | - $orderEntities[ $order->getNumber() ] = $order; | |
| 504 | + | |
| 505 | + $wcOrder = $this->getWcOrderById( (int) $orderId ); | |
| 506 | + assert( $wcOrder !== null, 'WC order has to be present' ); | |
| 507 | + | |
| 508 | + try { | |
| 509 | + $orderEntities[ $orderId ] = $this->builder->build( $wcOrder, $packeteryOrdersResult[ $orderId ] ); | |
| 510 | + } catch ( InvalidCarrierException $exception ) { | |
| 511 | + continue; | |
| 404 | 512 | } |
| 405 | 513 | } |
| 406 | 514 | |
| 407 | 515 | return $orderEntities; |
| @@ -407,87 +515,79 @@ | ||
| 407 | 515 | return $orderEntities; |
| 408 | 516 | } |
| 409 | 517 | |
| 410 | 518 | /** |
| 411 | - * Quote array of strings. | |
| 412 | - * | |
| 413 | - * @param array $input Input. | |
| 414 | - * | |
| 415 | - * @return array | |
| 416 | - */ | |
| 417 | - private function quoteArrayOfStrings( array $input ): array { | |
| 418 | - return array_map( | |
| 419 | - function ( string $item ) { | |
| 420 | - $wpdb = $this->wpdb; | |
| 421 | - return $wpdb->prepare( '%s', $item ); | |
| 422 | - }, | |
| 423 | - $input | |
| 424 | - ); | |
| 425 | - } | |
| 426 | - | |
| 427 | - /** | |
| 428 | 519 | * Finds orders. |
| 429 | 520 | * |
| 430 | - * @param array $allowedPacketStatuses Allowed packet statuses. | |
| 431 | - * @param array $allowedOrderStatuses Allowed order statuses. | |
| 432 | - * @param int $maxDays Max number of days of single packet sync. | |
| 433 | - * @param int $limit Number of records. | |
| 521 | + * @param string[] $allowedPacketStatuses Allowed packet statuses. | |
| 522 | + * @param string[] $allowedOrderStatuses Allowed order statuses. | |
| 523 | + * @param int $maxDays Max number of days of single packet sync. | |
| 524 | + * @param int $limit Number of records. | |
| 434 | 525 | * |
| 435 | - * @return iterable|Order[] | |
| 526 | + * @return \Generator<int> | |
| 527 | + * @throws Exception Exception. | |
| 436 | 528 | */ |
| 437 | - public function findStatusSyncingOrders( array $allowedPacketStatuses, array $allowedOrderStatuses, int $maxDays, int $limit ): iterable { | |
| 438 | - $wpdb = $this->wpdb; | |
| 439 | - $dateLimit = Helper::now()->modify( '- ' . $maxDays . ' days' )->format( 'Y-m-d H:i:s' ); | |
| 529 | + public function findStatusSyncingOrderIds( array $allowedPacketStatuses, array $allowedOrderStatuses, int $maxDays, int $limit ): \Generator { | |
| 530 | + $dateLimit = CoreHelper::now()->modify( '- ' . $maxDays . ' days' )->format( 'Y-m-d H:i:s' ); | |
| 440 | 531 | |
| 441 | - $andWhere = []; | |
| 532 | + $andWhere = [ '`o`.`packet_id` IS NOT NULL' ]; | |
| 533 | + $hposEnabled = ModuleHelper::isHposEnabled(); | |
| 442 | 534 | |
| 443 | - $andWhere[] = 'o.`packet_id` IS NOT NULL'; | |
| 444 | - $andWhere[] = $wpdb->prepare( 'wp_p.`post_date_gmt` >= %s', $dateLimit ); | |
| 535 | + if ( $hposEnabled ) { | |
| 536 | + $andWhere[] = $this->wpdbAdapter->prepare( '`wc_o`.`date_created_gmt` >= %s', $dateLimit ); | |
| 537 | + } else { | |
| 538 | + $andWhere[] = $this->wpdbAdapter->prepare( '`wp_p`.`post_date_gmt` >= %s', $dateLimit ); | |
| 539 | + } | |
| 445 | 540 | |
| 446 | 541 | $orPacketStatus = []; |
| 447 | - $orPacketStatus[] = 'o.`packet_status` IS NULL'; | |
| 542 | + $orPacketStatus[] = '`o`.`packet_status` IS NULL'; | |
| 448 | 543 | |
| 449 | - if ( $allowedPacketStatuses ) { | |
| 450 | - $orPacketStatus[] = 'o.`packet_status` IN (' . implode( ',', $this->quoteArrayOfStrings( $allowedPacketStatuses ) ) . ')'; | |
| 544 | + if ( count( $allowedPacketStatuses ) > 0 ) { | |
| 545 | + $orPacketStatus[] = '`o`.`packet_status` IN (' . $this->wpdbAdapter->prepareInClause( $allowedPacketStatuses ) . ')'; | |
| 451 | 546 | } |
| 452 | 547 | |
| 453 | - if ( $orPacketStatus ) { | |
| 548 | + if ( count( $orPacketStatus ) > 0 ) { | |
| 454 | 549 | $andWhere[] = '(' . implode( ' OR ', $orPacketStatus ) . ')'; |
| 455 | 550 | } |
| 456 | 551 | |
| 457 | - if ( $allowedOrderStatuses ) { | |
| 458 | - $andWhere[] = 'wp_p.`post_status` IN (' . implode( ',', $this->quoteArrayOfStrings( $allowedOrderStatuses ) ) . ')'; | |
| 552 | + if ( count( $allowedOrderStatuses ) > 0 && $hposEnabled ) { | |
| 553 | + $andWhere[] = '`wc_o`.`status` IN (' . $this->wpdbAdapter->prepareInClause( $allowedOrderStatuses ) . ')'; | |
| 554 | + } elseif ( count( $allowedOrderStatuses ) > 0 && $hposEnabled === false ) { | |
| 555 | + $andWhere[] = '`wp_p`.`post_status` IN (' . $this->wpdbAdapter->prepareInClause( $allowedOrderStatuses ) . ')'; | |
| 459 | 556 | } else { |
| 460 | 557 | $andWhere[] = '1 = 0'; |
| 461 | 558 | } |
| 462 | 559 | |
| 463 | 560 | $where = ''; |
| 464 | - if ( $andWhere ) { | |
| 561 | + if ( count( $andWhere ) > 0 ) { | |
| 465 | 562 | $where = ' WHERE ' . implode( ' AND ', $andWhere ); |
| 466 | 563 | } |
| 467 | 564 | |
| 468 | - // @codingStandardsIgnoreStart | |
| 469 | - $sql = $wpdb->prepare( | |
| 565 | + if ( $hposEnabled ) { | |
| 566 | + $orderBy = ' ORDER BY `wc_o`.`date_created_gmt` '; | |
| 567 | + } else { | |
| 568 | + $orderBy = ' ORDER BY `wp_p`.`post_date_gmt` '; | |
| 569 | + } | |
| 570 | + | |
| 571 | + $sql = $this->wpdbAdapter->prepare( | |
| 470 | 572 | ' |
| 471 | - SELECT o.* FROM `' . $wpdb->packetery_order . '` o | |
| 472 | - JOIN `' . $wpdb->posts . '` wp_p ON wp_p.`ID` = o.`id` | |
| 573 | + SELECT `o`.`id` FROM `' . $this->wpdbAdapter->packeteryOrder . '` `o` | |
| 574 | + ' . $this->getWcOrderJoinClause() . ' | |
| 473 | 575 | ' . $where . ' |
| 474 | - ORDER BY wp_p.`post_date` | |
| 576 | + ' . $orderBy . ' | |
| 475 | 577 | LIMIT %d', |
| 476 | 578 | $limit |
| 477 | 579 | ); |
| 478 | 580 | |
| 479 | - $rows = $wpdb->get_results( $sql ); | |
| 480 | - // @codingStandardsIgnoreEnd | |
| 581 | + $rows = $this->wpdbAdapter->get_results( $sql ); | |
| 481 | 582 | |
| 482 | 583 | foreach ( $rows as $row ) { |
| 483 | - $wcOrder = wc_get_order( $row->id ); | |
| 484 | - if ( false === $wcOrder || ! $wcOrder->has_shipping_method( ShippingMethod::PACKETERY_METHOD_ID ) ) { | |
| 584 | + $wcOrder = $this->getWcOrderById( (int) $row->id ); | |
| 585 | + if ( $wcOrder === null || ! ShippingProvider::wcOrderHasOurMethod( $wcOrder ) ) { | |
| 485 | 586 | continue; |
| 486 | 587 | } |
| 487 | 588 | |
| 488 | - $partial = $this->createPartialOrder( $row ); | |
| 489 | - yield $this->builder->finalize( $wcOrder, $partial ); | |
| 589 | + yield (int) $row->id; | |
| 490 | 590 | } |
| 491 | 591 | } |
| 492 | 592 | |
| 493 | 593 | /** |
| @@ -495,15 +595,9 @@ | ||
| 495 | 595 | * |
| 496 | 596 | * @return int |
| 497 | 597 | */ |
| 498 | 598 | public function countOrdersToSubmit(): int { |
| 499 | - $wpdb = $this->wpdb; | |
| 500 | - | |
| 501 | - return (int) $wpdb->get_var( | |
| 502 | - 'SELECT COUNT(DISTINCT o.id) FROM `' . $wpdb->packetery_order . '` o | |
| 503 | - JOIN `' . $wpdb->posts . '` wp_p ON wp_p.`ID` = o.`id` | |
| 504 | - WHERE o.`carrier_id` IS NOT NULL AND o.`is_exported` = false' | |
| 505 | - ); | |
| 599 | + return $this->countOrders( [ 'packetery_to_submit' => '1' ] ); | |
| 506 | 600 | } |
| 507 | 601 | |
| 508 | 602 | /** |
| 509 | 603 | * Counts order to be printed. |
| @@ -510,89 +604,151 @@ | ||
| 510 | 604 | * |
| 511 | 605 | * @return int |
| 512 | 606 | */ |
| 513 | 607 | public function countOrdersToPrint(): int { |
| 514 | - $wpdb = $this->wpdb; | |
| 515 | - | |
| 516 | - return (int) $wpdb->get_var( | |
| 517 | - 'SELECT COUNT(DISTINCT o.id) FROM `' . $wpdb->packetery_order . '` o | |
| 518 | - JOIN `' . $wpdb->posts . '` wp_p ON wp_p.`ID` = o.`id` | |
| 519 | - WHERE o.`packet_id` IS NOT NULL AND o.`is_label_printed` = false' | |
| 520 | - ); | |
| 608 | + return $this->countOrders( [ 'packetery_to_print' => '1' ] ); | |
| 521 | 609 | } |
| 522 | 610 | |
| 523 | 611 | /** |
| 524 | - * Deletes all custom table records linked to permanently deleted orders. | |
| 612 | + * Counts orders by given params. | |
| 525 | 613 | * |
| 526 | - * @return void | |
| 614 | + * @param array<string, string> $params Params. | |
| 615 | + * | |
| 616 | + * @return int | |
| 527 | 617 | */ |
| 528 | - public function deleteOrphans(): void { | |
| 529 | - $wpdb = $this->wpdb; | |
| 618 | + private function countOrders( array $params ): int { | |
| 619 | + $clauses = [ 'join' => '' ]; | |
| 530 | 620 | |
| 531 | - $wpdb->query( | |
| 532 | - 'DELETE `' . $wpdb->packetery_order . '` FROM `' . $wpdb->packetery_order . '` | |
| 533 | - LEFT JOIN `' . $wpdb->posts . '` ON `' . $wpdb->posts . '`.`ID` = `' . $wpdb->packetery_order . '`.`id` | |
| 534 | - WHERE `' . $wpdb->posts . '`.`ID` IS NULL' | |
| 621 | + if ( ModuleHelper::isHposEnabled() ) { | |
| 622 | + $clauses['select'] = ' SELECT COUNT(DISTINCT `' . $this->wpdbAdapter->wcOrders . '`.`id`) '; | |
| 623 | + $clauses['from'] = ' FROM `' . $this->wpdbAdapter->wcOrders . '` '; | |
| 624 | + } else { | |
| 625 | + $clauses['select'] = ' SELECT COUNT(DISTINCT `' . $this->wpdbAdapter->posts . '`.`ID`) '; | |
| 626 | + $clauses['from'] = ' FROM `' . $this->wpdbAdapter->posts . '` '; | |
| 627 | + } | |
| 628 | + | |
| 629 | + $clauses['where'] = ' WHERE `' . $this->wpdbAdapter->packeteryOrder . '`.`id` IS NOT NULL '; | |
| 630 | + $clauses = $this->processClauses( $clauses, null, $params ); | |
| 631 | + | |
| 632 | + return (int) $this->wpdbAdapter->get_var( | |
| 633 | + $clauses['select'] . | |
| 634 | + $clauses['from'] . | |
| 635 | + $clauses['join'] . | |
| 636 | + $clauses['where'] | |
| 535 | 637 | ); |
| 536 | 638 | } |
| 537 | 639 | |
| 538 | 640 | /** |
| 539 | - * Adds adult content column. | |
| 641 | + * Gets WC Order join clause. | |
| 540 | 642 | * |
| 541 | - * @return void | |
| 643 | + * @return string | |
| 542 | 644 | */ |
| 543 | - public function addAdultContentColumn(): void { | |
| 544 | - $wpdb = $this->wpdb; | |
| 645 | + public function getWcOrderJoinClause(): string { | |
| 646 | + $packeteryTableAlias = 'o'; | |
| 545 | 647 | |
| 546 | - $wpdb->query( 'ALTER TABLE `' . $wpdb->packetery_order . '` ADD COLUMN `adult_content` boolean NULL DEFAULT NULL AFTER `height`' ); | |
| 648 | + if ( ModuleHelper::isHposEnabled() ) { | |
| 649 | + $sourceTableAlias = 'wc_o'; | |
| 650 | + | |
| 651 | + return sprintf( | |
| 652 | + 'JOIN `%s` `%s` ON `%s`.`id` = `%s`.`id`', | |
| 653 | + $this->wpdbAdapter->wcOrders, | |
| 654 | + $sourceTableAlias, | |
| 655 | + $sourceTableAlias, | |
| 656 | + $packeteryTableAlias | |
| 657 | + ); | |
| 658 | + } | |
| 659 | + | |
| 660 | + $sourceTableAlias = 'wp_p'; | |
| 661 | + | |
| 662 | + return sprintf( | |
| 663 | + 'JOIN `%s` `%s` ON `%s`.`ID` = `%s`.`id`', | |
| 664 | + $this->wpdbAdapter->posts, | |
| 665 | + $sourceTableAlias, | |
| 666 | + $sourceTableAlias, | |
| 667 | + $packeteryTableAlias | |
| 668 | + ); | |
| 547 | 669 | } |
| 548 | 670 | |
| 549 | 671 | /** |
| 550 | - * Adds value column. | |
| 672 | + * Deletes all custom table records linked to permanently deleted orders. | |
| 551 | 673 | * |
| 552 | 674 | * @return void |
| 553 | 675 | */ |
| 554 | - public function addValueColumn(): void { | |
| 555 | - $wpdb = $this->wpdb; | |
| 676 | + public function deleteOrphans(): void { | |
| 677 | + if ( ModuleHelper::isHposEnabled() ) { | |
| 678 | + $this->wpdbAdapter->query( | |
| 679 | + 'DELETE `' . $this->wpdbAdapter->packeteryOrder . '` FROM `' . $this->wpdbAdapter->packeteryOrder . '` | |
| 680 | + LEFT JOIN `' . $this->wpdbAdapter->wcOrders . '` ON `' . $this->wpdbAdapter->wcOrders . '`.`id` = `' . $this->wpdbAdapter->packeteryOrder . '`.`id` | |
| 681 | + WHERE `' . $this->wpdbAdapter->wcOrders . '`.`id` IS NULL' | |
| 682 | + ); | |
| 556 | 683 | |
| 557 | - $wpdb->query( 'ALTER TABLE `' . $wpdb->packetery_order . '` ADD COLUMN `value` double NULL DEFAULT NULL AFTER `adult_content`' ); | |
| 684 | + return; | |
| 685 | + } | |
| 686 | + | |
| 687 | + $this->wpdbAdapter->query( | |
| 688 | + 'DELETE `' . $this->wpdbAdapter->packeteryOrder . '` FROM `' . $this->wpdbAdapter->packeteryOrder . '` | |
| 689 | + LEFT JOIN `' . $this->wpdbAdapter->posts . '` ON `' . $this->wpdbAdapter->posts . '`.`ID` = `' . $this->wpdbAdapter->packeteryOrder . '`.`id` | |
| 690 | + WHERE `' . $this->wpdbAdapter->posts . '`.`ID` IS NULL' | |
| 691 | + ); | |
| 558 | 692 | } |
| 559 | 693 | |
| 560 | 694 | /** |
| 561 | - * Adds COD column. | |
| 695 | + * Deletes order data including customs declaration and its items from custom tables. | |
| 562 | 696 | * |
| 563 | - * @return void | |
| 697 | + * @param int $orderId Order id. | |
| 698 | + * | |
| 699 | + * @return bool true on success, false in case of db failure. | |
| 564 | 700 | */ |
| 565 | - public function addCodColumn(): void { | |
| 566 | - $wpdb = $this->wpdb; | |
| 701 | + public function delete( int $orderId ): bool { | |
| 702 | + try { | |
| 703 | + $this->customsDeclarationRepository->delete( (string) $orderId ); | |
| 704 | + $this->wpdbAdapter->delete( $this->wpdbAdapter->packeteryOrder, [ 'id' => $orderId ], '%d' ); | |
| 567 | 705 | |
| 568 | - $wpdb->query( 'ALTER TABLE `' . $wpdb->packetery_order . '` ADD COLUMN `cod` double NULL DEFAULT NULL AFTER `value`' ); | |
| 706 | + return true; | |
| 707 | + } catch ( DeleteErrorException $e ) { | |
| 708 | + return false; | |
| 709 | + } | |
| 569 | 710 | } |
| 570 | 711 | |
| 571 | 712 | /** |
| 572 | - * Deletes data from custom table. | |
| 713 | + * Fires after post deletion. | |
| 573 | 714 | * |
| 574 | - * @param int $orderId Order id. | |
| 715 | + * @param int|mixed $postId Post id. | |
| 716 | + * @param WP_Post|mixed $post Post object. | |
| 575 | 717 | * |
| 576 | 718 | * @return void |
| 577 | 719 | */ |
| 578 | - private function delete( int $orderId ): void { | |
| 579 | - $wpdb = $this->wpdb; | |
| 720 | + public function deletedPostHook( $postId, $post ): void { | |
| 721 | + if ( ! is_int( $postId ) ) { | |
| 722 | + WcLogger::logArgumentTypeError( __METHOD__, 'postId', 'int', $postId ); | |
| 580 | 723 | |
| 581 | - $wpdb->delete( $wpdb->packetery_order, [ 'id' => $orderId ], '%d' ); | |
| 724 | + return; | |
| 725 | + } | |
| 726 | + | |
| 727 | + if ( ! $post instanceof WP_Post ) { | |
| 728 | + WcLogger::logArgumentTypeError( __METHOD__, 'post', WP_Post::class, $post ); | |
| 729 | + | |
| 730 | + return; | |
| 731 | + } | |
| 732 | + | |
| 733 | + // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps | |
| 734 | + if ( $post->post_type === 'shop_order' ) { | |
| 735 | + $this->delete( $postId ); | |
| 736 | + } | |
| 582 | 737 | } |
| 583 | 738 | |
| 584 | 739 | /** |
| 585 | - * Fires after post deletion. | |
| 740 | + * Gets WC_Order object. | |
| 586 | 741 | * |
| 587 | - * @param int $postId Post id. | |
| 588 | - * @param WP_Post $post Post object. | |
| 742 | + * @param int $id Order id. | |
| 589 | 743 | * |
| 590 | - * @return void | |
| 744 | + * @return WC_Order|null | |
| 591 | 745 | */ |
| 592 | - public function deletedPostHook( int $postId, WP_Post $post ): void { | |
| 593 | - if ( 'shop_order' === $post->post_type ) { | |
| 594 | - $this->delete( $postId ); | |
| 746 | + public function getWcOrderById( int $id ): ?WC_Order { | |
| 747 | + $result = wc_get_order( $id ); | |
| 748 | + if ( $result instanceof WC_Order ) { | |
| 749 | + return $result; | |
| 595 | 750 | } |
| 751 | + | |
| 752 | + return null; | |
| 596 | 753 | } |
| 597 | - | |
| 598 | 754 | } |
