diff --git a/lib/API/JSON/ResponseBuilders/CustomFieldsResponseBuilder.php b/lib/API/JSON/ResponseBuilders/CustomFieldsResponseBuilder.php index 0f5ac303de..c081220a87 100644 --- a/lib/API/JSON/ResponseBuilders/CustomFieldsResponseBuilder.php +++ b/lib/API/JSON/ResponseBuilders/CustomFieldsResponseBuilder.php @@ -7,7 +7,7 @@ use MailPoet\Entities\CustomFieldEntity; class CustomFieldsResponseBuilder { /** - * @param CustomFieldEntity[] $custom_fields + * @param CustomFieldEntity[] $customFields * @return array */ public function buildBatch(array $customFields) { @@ -15,7 +15,7 @@ class CustomFieldsResponseBuilder { } /** - * @param CustomFieldEntity $custom_field + * @param CustomFieldEntity $customField * @return array */ public function build(CustomFieldEntity $customField) { diff --git a/lib/Config/MP2Migrator.php b/lib/Config/MP2Migrator.php index 2a6ec63693..65162d1a07 100644 --- a/lib/Config/MP2Migrator.php +++ b/lib/Config/MP2Migrator.php @@ -379,7 +379,7 @@ class MP2Migrator { /** * Import a segment * - * @param array $list_data List data + * @param array $listData List data * @return Segment */ private function importSegment($listData) { @@ -453,7 +453,7 @@ class MP2Migrator { /** * Import a custom field * - * @param array $custom_field MP2 custom field + * @param array $customField MP2 custom field * @return CustomField */ private function importCustomField($customField) { @@ -471,7 +471,7 @@ class MP2Migrator { /** * Map the MailPoet 2 custom field type with the MailPoet custom field type * - * @param string $mp2_type MP2 custom field type + * @param string $mp2Type MP2 custom field type * @return string MP3 custom field type */ private function mapCustomFieldType($mp2Type) { @@ -536,7 +536,7 @@ class MP2Migrator { /** * Map the validate value * - * @param string $mp2_value MP2 value + * @param string $mp2Value MP2 value * @return string MP3 value */ private function mapCustomFieldValidateValue($mp2Value) { @@ -616,7 +616,7 @@ class MP2Migrator { /** * Import a subscriber * - * @param array $user_data User data + * @param array $userData User data * @return Subscriber */ private function importSubscriber($userData) { @@ -649,7 +649,7 @@ class MP2Migrator { /** * Map the MailPoet 2 user status with MailPoet 3 * - * @param int $mp2_user_status MP2 user status + * @param int $mp2UserStatus MP2 user status * @return string MP3 user status */ private function mapUserStatus($mp2UserStatus) { @@ -678,7 +678,7 @@ class MP2Migrator { * Import the segments for a subscriber * * @param Subscriber $subscriber MP3 subscriber - * @param int $user_id MP2 user ID + * @param int $userId MP2 user ID */ private function importSubscriberSegments($subscriber, $userId) { $userLists = $this->getUserLists($userId); @@ -691,7 +691,7 @@ class MP2Migrator { * Get the lists for a user * * @global object $wpdb - * @param int $user_id User ID + * @param int $userId User ID * @return array Users Lists */ private function getUserLists($userId) { @@ -711,8 +711,8 @@ class MP2Migrator { /** * Import a subscriber segment * - * @param int $subscriber_id - * @param array $user_list + * @param int $subscriberId + * @param array $userList * @return SubscriberSegment|null */ private function importSubscriberSegment($subscriberId, $userList) { @@ -769,9 +769,9 @@ class MP2Migrator { /** * Import a subscriber custom field * - * @param int $subscriber_id Subscriber ID - * @param array $custom_field Custom field - * @param string $custom_field_value Custom field value + * @param int $subscriberId Subscriber ID + * @param array $customField Custom field + * @param string $customFieldValue Custom field value * @return SubscriberCustomField */ private function importSubscriberCustomField($subscriberId, $customField, $customFieldValue) { @@ -864,7 +864,7 @@ class MP2Migrator { /** * Import a form * - * @param array $form_data Form data + * @param array $formData Form data * @return Form */ private function importForm($formData) { @@ -928,7 +928,7 @@ class MP2Migrator { /** * Get the MP3 segments IDs of the MP2 lists IDs * - * @param array $mp2_list_ids + * @param array $mp2ListIds */ private function getMappedSegmentIds($mp2ListIds) { $mp3SegmentIds = []; @@ -1105,7 +1105,7 @@ class MP2Migrator { * Get an email * * @global object $wpdb - * @param int $email_id + * @param int $emailId * @return array Email */ private function getEmail($emailId) { @@ -1126,7 +1126,7 @@ class MP2Migrator { /** * Map the Email frequency interval * - * @param string $interval_str Interval + * @param string $intervalStr Interval * @return string Interval */ private function mapFrequencyInterval($intervalStr) { @@ -1156,8 +1156,8 @@ class MP2Migrator { /** * Map the Email frequency number * - * @param int $emails_number Emails number - * @param string $interval_str Interval + * @param int $emailsNumber Emails number + * @param string $intervalStr Interval * @return int Emails number */ private function mapFrequencyEmails($emailsNumber, $intervalStr) { diff --git a/lib/Cron/DaemonHttpRunner.php b/lib/Cron/DaemonHttpRunner.php index df01424068..ceafb3ac3b 100644 --- a/lib/Cron/DaemonHttpRunner.php +++ b/lib/Cron/DaemonHttpRunner.php @@ -125,7 +125,7 @@ class DaemonHttpRunner { } /** - * @param array|null $settings_daemon_data + * @param array|null $settingsDaemonData * * @return boolean */ diff --git a/lib/Cron/Workers/StatsNotifications/NewsletterLinkRepository.php b/lib/Cron/Workers/StatsNotifications/NewsletterLinkRepository.php index 0321486a4c..f658402615 100644 --- a/lib/Cron/Workers/StatsNotifications/NewsletterLinkRepository.php +++ b/lib/Cron/Workers/StatsNotifications/NewsletterLinkRepository.php @@ -11,7 +11,7 @@ class NewsletterLinkRepository extends Repository { } /** - * @param int $newsletter_id + * @param int $newsletterId * @return NewsletterLinkEntity|null */ public function findTopLinkForNewsletter($newsletterId) { diff --git a/lib/Cron/Workers/StatsNotifications/StatsNotificationsRepository.php b/lib/Cron/Workers/StatsNotifications/StatsNotificationsRepository.php index 2e5dc32c65..7262df588c 100644 --- a/lib/Cron/Workers/StatsNotifications/StatsNotificationsRepository.php +++ b/lib/Cron/Workers/StatsNotifications/StatsNotificationsRepository.php @@ -14,7 +14,7 @@ class StatsNotificationsRepository extends Repository { } /** - * @param int $newsletter_id + * @param int $newsletterId * @return StatsNotificationEntity|null */ public function findOneByNewsletterId($newsletterId) { diff --git a/lib/CustomFields/CustomFieldsRepository.php b/lib/CustomFields/CustomFieldsRepository.php index 2b00cc6776..383cffef28 100644 --- a/lib/CustomFields/CustomFieldsRepository.php +++ b/lib/CustomFields/CustomFieldsRepository.php @@ -6,9 +6,9 @@ use MailPoet\Doctrine\Repository; use MailPoet\Entities\CustomFieldEntity; /** - * @method CustomFieldEntity[] findBy(array $criteria, array $order_by = null, int $limit = null, int $offset = null) + * @method CustomFieldEntity[] findBy(array $criteria, array $orderBy = null, int $limit = null, int $offset = null) * @method CustomFieldEntity[] findAll() - * @method CustomFieldEntity|null findOneBy(array $criteria, array $order_by = null) + * @method CustomFieldEntity|null findOneBy(array $criteria, array $orderBy = null) * @method CustomFieldEntity|null findOneById(mixed $id) * @method void persist(CustomFieldEntity $entity) * @method void remove(CustomFieldEntity $entity) diff --git a/lib/Doctrine/EntityTraits/DeletedAtTrait.php b/lib/Doctrine/EntityTraits/DeletedAtTrait.php index ef7946a986..990ee40e78 100644 --- a/lib/Doctrine/EntityTraits/DeletedAtTrait.php +++ b/lib/Doctrine/EntityTraits/DeletedAtTrait.php @@ -17,7 +17,7 @@ trait DeletedAtTrait { return $this->deletedAt; } - /** @param DateTimeInterface|null $deleted_at */ + /** @param DateTimeInterface|null $deletedAt */ public function setDeletedAt($deletedAt) { $this->deletedAt = $deletedAt; } diff --git a/lib/Doctrine/Repository.php b/lib/Doctrine/Repository.php index 8d6b4ef22f..9111dad2aa 100644 --- a/lib/Doctrine/Repository.php +++ b/lib/Doctrine/Repository.php @@ -25,7 +25,7 @@ abstract class Repository { /** * @param array $criteria - * @param array|null $order_by + * @param array|null $orderBy * @param int|null $limit * @param int|null $offset * @return array @@ -36,7 +36,7 @@ abstract class Repository { /** * @param array $criteria - * @param array|null $order_by + * @param array|null $orderBy * @return object|null */ public function findOneBy(array $criteria, array $orderBy = null) { diff --git a/lib/DynamicSegments/Filters/EmailAction.php b/lib/DynamicSegments/Filters/EmailAction.php index 0d4090c4c1..994e7ade92 100644 --- a/lib/DynamicSegments/Filters/EmailAction.php +++ b/lib/DynamicSegments/Filters/EmailAction.php @@ -37,8 +37,8 @@ class EmailAction implements Filter { private $connect; /** - * @param int $newsletter_id - * @param int $link_id + * @param int $newsletterId + * @param int $linkId * @param string $action * @param string $connect */ diff --git a/lib/DynamicSegments/Filters/WooCommerceCategory.php b/lib/DynamicSegments/Filters/WooCommerceCategory.php index 79d5bf0417..67a00c182a 100644 --- a/lib/DynamicSegments/Filters/WooCommerceCategory.php +++ b/lib/DynamicSegments/Filters/WooCommerceCategory.php @@ -19,7 +19,7 @@ class WooCommerceCategory implements Filter { private $connect; /** - * @param int $category_id + * @param int $categoryId * @param string $connect */ public function __construct($categoryId, $connect = null) { diff --git a/lib/DynamicSegments/Filters/WooCommerceProduct.php b/lib/DynamicSegments/Filters/WooCommerceProduct.php index fadaecc70a..d449f1781a 100644 --- a/lib/DynamicSegments/Filters/WooCommerceProduct.php +++ b/lib/DynamicSegments/Filters/WooCommerceProduct.php @@ -18,7 +18,7 @@ class WooCommerceProduct implements Filter { private $connect; /** - * @param int $product_id + * @param int $productId * @param string $connect */ public function __construct($productId, $connect = null) { diff --git a/lib/DynamicSegments/FreePluginConnectors/AddToNewslettersSegments.php b/lib/DynamicSegments/FreePluginConnectors/AddToNewslettersSegments.php index 4566a065ba..b7822b450e 100644 --- a/lib/DynamicSegments/FreePluginConnectors/AddToNewslettersSegments.php +++ b/lib/DynamicSegments/FreePluginConnectors/AddToNewslettersSegments.php @@ -19,7 +19,7 @@ class AddToNewslettersSegments { } /** - * @param array $initial_segments + * @param array $initialSegments * * @return array */ diff --git a/lib/DynamicSegments/FreePluginConnectors/AddToSubscribersFilters.php b/lib/DynamicSegments/FreePluginConnectors/AddToSubscribersFilters.php index d9c3cc3beb..d952912b8f 100644 --- a/lib/DynamicSegments/FreePluginConnectors/AddToSubscribersFilters.php +++ b/lib/DynamicSegments/FreePluginConnectors/AddToSubscribersFilters.php @@ -19,7 +19,7 @@ class AddToSubscribersFilters { } /** - * @param array $segment_filters + * @param array $segmentFilters * * @return array */ diff --git a/lib/DynamicSegments/FreePluginConnectors/SendingNewslettersSubscribersFinder.php b/lib/DynamicSegments/FreePluginConnectors/SendingNewslettersSubscribersFinder.php index 0cd5fc4372..6e1030ce64 100644 --- a/lib/DynamicSegments/FreePluginConnectors/SendingNewslettersSubscribersFinder.php +++ b/lib/DynamicSegments/FreePluginConnectors/SendingNewslettersSubscribersFinder.php @@ -23,7 +23,7 @@ class SendingNewslettersSubscribersFinder { /** * @param Segment $segment - * @param int[] $subscribers_to_process_ids + * @param int[] $subscribersToProcessIds * * @return Subscriber[] */ diff --git a/lib/DynamicSegments/Mappers/DBMapper.php b/lib/DynamicSegments/Mappers/DBMapper.php index bd14ce329f..4ac4e975f4 100644 --- a/lib/DynamicSegments/Mappers/DBMapper.php +++ b/lib/DynamicSegments/Mappers/DBMapper.php @@ -14,8 +14,8 @@ use MailPoet\Models\DynamicSegmentFilter; class DBMapper { /** - * @param DynamicSegment $segment_data - * @param DynamicSegmentFilter[] $filters_data + * @param DynamicSegment $segmentData + * @param DynamicSegmentFilter[] $filtersData * * @return DynamicSegment */ @@ -26,8 +26,8 @@ class DBMapper { } /** - * @param DynamicSegment[] $segments_data - * @param DynamicSegmentFilter[] $filters_data + * @param DynamicSegment[] $segmentsData + * @param DynamicSegmentFilter[] $filtersData * * @return DynamicSegment[] */ diff --git a/lib/DynamicSegments/Persistence/Loading/SingleSegmentLoader.php b/lib/DynamicSegments/Persistence/Loading/SingleSegmentLoader.php index 722ff29691..905f17b3a5 100644 --- a/lib/DynamicSegments/Persistence/Loading/SingleSegmentLoader.php +++ b/lib/DynamicSegments/Persistence/Loading/SingleSegmentLoader.php @@ -15,7 +15,7 @@ class SingleSegmentLoader { } /** - * @param string|int $segment_id + * @param string|int $segmentId * @return DynamicSegment */ public function load($segmentId) { diff --git a/lib/DynamicSegments/Persistence/Loading/SubscribersCount.php b/lib/DynamicSegments/Persistence/Loading/SubscribersCount.php index a2429d2937..507763cf0a 100644 --- a/lib/DynamicSegments/Persistence/Loading/SubscribersCount.php +++ b/lib/DynamicSegments/Persistence/Loading/SubscribersCount.php @@ -20,7 +20,7 @@ class SubscribersCount { } /** - * @param DynamicSegment $dynamic_segment + * @param DynamicSegment $dynamicSegment * * @return int */ diff --git a/lib/DynamicSegments/Persistence/Loading/SubscribersIds.php b/lib/DynamicSegments/Persistence/Loading/SubscribersIds.php index 02282d3202..245c1450ef 100644 --- a/lib/DynamicSegments/Persistence/Loading/SubscribersIds.php +++ b/lib/DynamicSegments/Persistence/Loading/SubscribersIds.php @@ -22,8 +22,8 @@ class SubscribersIds { /** * Finds subscribers in a dynamic segment and returns their ids. * - * @param DynamicSegment $dynamic_segment - * @param array $limit_to_subscribers_ids If passed the result will be limited only to ids within this array + * @param DynamicSegment $dynamicSegment + * @param array $limitToSubscribersIds If passed the result will be limited only to ids within this array * * @return Subscriber[] */ diff --git a/lib/Entities/NewsletterEntity.php b/lib/Entities/NewsletterEntity.php index ac637fa12b..f2b4d786de 100644 --- a/lib/Entities/NewsletterEntity.php +++ b/lib/Entities/NewsletterEntity.php @@ -199,7 +199,7 @@ class NewsletterEntity { } /** - * @param string $sender_address + * @param string $senderAddress */ public function setSenderAddress($senderAddress) { $this->senderAddress = $senderAddress; @@ -213,7 +213,7 @@ class NewsletterEntity { } /** - * @param string $sender_name + * @param string $senderName */ public function setSenderName($senderName) { $this->senderName = $senderName; @@ -241,7 +241,7 @@ class NewsletterEntity { } /** - * @param string $reply_to_address + * @param string $replyToAddress */ public function setReplyToAddress($replyToAddress) { $this->replyToAddress = $replyToAddress; @@ -255,7 +255,7 @@ class NewsletterEntity { } /** - * @param string $reply_to_name + * @param string $replyToName */ public function setReplyToName($replyToName) { $this->replyToName = $replyToName; @@ -297,7 +297,7 @@ class NewsletterEntity { } /** - * @param DateTimeInterface|null $sent_at + * @param DateTimeInterface|null $sentAt */ public function setSentAt($sentAt) { $this->sentAt = $sentAt; @@ -318,7 +318,7 @@ class NewsletterEntity { } /** - * @param string $ga_campaign + * @param string $gaCampaign */ public function setGaCampaign($gaCampaign) { $this->gaCampaign = $gaCampaign; @@ -326,7 +326,7 @@ class NewsletterEntity { /** - * @param string|null $unsubscribe_token + * @param string|null $unsubscribeToken */ public function setUnsubscribeToken($unsubscribeToken) { $this->unsubscribeToken = $unsubscribeToken; diff --git a/lib/Entities/NewsletterOptionEntity.php b/lib/Entities/NewsletterOptionEntity.php index 92cb2375b6..89b327276e 100644 --- a/lib/Entities/NewsletterOptionEntity.php +++ b/lib/Entities/NewsletterOptionEntity.php @@ -70,7 +70,7 @@ class NewsletterOptionEntity { } /** - * @param NewsletterOptionFieldEntity $option_field + * @param NewsletterOptionFieldEntity $optionField */ public function setOptionField($optionField) { $this->optionField = $optionField; diff --git a/lib/Entities/NewsletterOptionFieldEntity.php b/lib/Entities/NewsletterOptionFieldEntity.php index 2283ddec92..dc396bdbb3 100644 --- a/lib/Entities/NewsletterOptionFieldEntity.php +++ b/lib/Entities/NewsletterOptionFieldEntity.php @@ -53,7 +53,7 @@ class NewsletterOptionFieldEntity { } /** - * @param string $newsletter_type + * @param string $newsletterType */ public function setNewsletterType($newsletterType) { $this->newsletterType = $newsletterType; diff --git a/lib/Entities/ScheduledTaskEntity.php b/lib/Entities/ScheduledTaskEntity.php index 2be48a5e05..9671d65c4b 100644 --- a/lib/Entities/ScheduledTaskEntity.php +++ b/lib/Entities/ScheduledTaskEntity.php @@ -113,7 +113,7 @@ class ScheduledTaskEntity { } /** - * @param DateTimeInterface|null $scheduled_at + * @param DateTimeInterface|null $scheduledAt */ public function setScheduledAt($scheduledAt) { $this->scheduledAt = $scheduledAt; @@ -127,7 +127,7 @@ class ScheduledTaskEntity { } /** - * @param DateTimeInterface|null $processed_at + * @param DateTimeInterface|null $processedAt */ public function setProcessedAt($processedAt) { $this->processedAt = $processedAt; diff --git a/lib/Entities/SendingQueueEntity.php b/lib/Entities/SendingQueueEntity.php index bbc1f0ca18..98ba09aa0b 100644 --- a/lib/Entities/SendingQueueEntity.php +++ b/lib/Entities/SendingQueueEntity.php @@ -89,7 +89,7 @@ class SendingQueueEntity { } /** - * @param array|null $newsletter_rendered_body + * @param array|null $newsletterRenderedBody */ public function setNewsletterRenderedBody($newsletterRenderedBody) { $this->newsletterRenderedBody = $newsletterRenderedBody; @@ -103,7 +103,7 @@ class SendingQueueEntity { } /** - * @param string|null $newsletter_rendered_subject + * @param string|null $newsletterRenderedSubject */ public function setNewsletterRenderedSubject($newsletterRenderedSubject) { $this->newsletterRenderedSubject = $newsletterRenderedSubject; @@ -131,7 +131,7 @@ class SendingQueueEntity { } /** - * @param int $count_total + * @param int $countTotal */ public function setCountTotal($countTotal) { $this->countTotal = $countTotal; @@ -145,7 +145,7 @@ class SendingQueueEntity { } /** - * @param int $count_processed + * @param int $countProcessed */ public function setCountProcessed($countProcessed) { $this->countProcessed = $countProcessed; @@ -159,7 +159,7 @@ class SendingQueueEntity { } /** - * @param int $count_to_process + * @param int $countToProcess */ public function setCountToProcess($countToProcess) { $this->countToProcess = $countToProcess; diff --git a/lib/Entities/SubscriberEntity.php b/lib/Entities/SubscriberEntity.php index b8ed8352eb..a07a7fba9a 100644 --- a/lib/Entities/SubscriberEntity.php +++ b/lib/Entities/SubscriberEntity.php @@ -124,7 +124,7 @@ class SubscriberEntity { } /** - * @param int|null $wp_user_id + * @param int|null $wpUserId */ public function setWpUserId($wpUserId) { $this->wpUserId = $wpUserId; @@ -138,7 +138,7 @@ class SubscriberEntity { } /** - * @param bool $is_woocommerce_user + * @param bool $isWoocommerceUser */ public function setIsWoocommerceUser($isWoocommerceUser) { $this->isWoocommerceUser = $isWoocommerceUser; @@ -152,7 +152,7 @@ class SubscriberEntity { } /** - * @param string $first_name + * @param string $firstName */ public function setFirstName($firstName) { $this->firstName = $firstName; @@ -166,7 +166,7 @@ class SubscriberEntity { } /** - * @param string $last_name + * @param string $lastName */ public function setLastName($lastName) { $this->lastName = $lastName; @@ -217,7 +217,7 @@ class SubscriberEntity { } /** - * @param string $subscribed_ip + * @param string $subscribedIp */ public function setSubscribedIp($subscribedIp) { $this->subscribedIp = $subscribedIp; @@ -231,7 +231,7 @@ class SubscriberEntity { } /** - * @param string|null $confirmed_ip + * @param string|null $confirmedIp */ public function setConfirmedIp($confirmedIp) { $this->confirmedIp = $confirmedIp; @@ -245,7 +245,7 @@ class SubscriberEntity { } /** - * @param DateTimeInterface|null $confirmed_at + * @param DateTimeInterface|null $confirmedAt */ public function setConfirmedAt($confirmedAt) { $this->confirmedAt = $confirmedAt; @@ -259,7 +259,7 @@ class SubscriberEntity { } /** - * @param DateTimeInterface|null $last_subscribed_at + * @param DateTimeInterface|null $lastSubscribedAt */ public function setLastSubscribedAt($lastSubscribedAt) { $this->lastSubscribedAt = $lastSubscribedAt; @@ -273,7 +273,7 @@ class SubscriberEntity { } /** - * @param string|null $unconfirmed_data + * @param string|null $unconfirmedData */ public function setUnconfirmedData($unconfirmedData) { $this->unconfirmedData = $unconfirmedData; @@ -313,7 +313,7 @@ class SubscriberEntity { } /** - * @param int $count_confirmations + * @param int $countConfirmations */ public function setConfirmationsCount($countConfirmations) { $this->countConfirmations = $countConfirmations; @@ -327,7 +327,7 @@ class SubscriberEntity { } /** - * @param string|null $unsubscribe_token + * @param string|null $unsubscribeToken */ public function setUnsubscribeToken($unsubscribeToken) { $this->unsubscribeToken = $unsubscribeToken; @@ -341,7 +341,7 @@ class SubscriberEntity { } /** - * @param string|null $link_token + * @param string|null $linkToken */ public function setLinkToken($linkToken) { $this->linkToken = $linkToken; diff --git a/lib/Entities/UserFlagEntity.php b/lib/Entities/UserFlagEntity.php index bfb81196ad..f9d4833ca5 100644 --- a/lib/Entities/UserFlagEntity.php +++ b/lib/Entities/UserFlagEntity.php @@ -39,7 +39,7 @@ class UserFlagEntity { return $this->userId; } - /** @param int $user_id */ + /** @param int $userId */ public function setUserId($userId) { $this->userId = $userId; } diff --git a/lib/Features/FeatureFlagsRepository.php b/lib/Features/FeatureFlagsRepository.php index f4f0d8790f..c70fbd19a4 100644 --- a/lib/Features/FeatureFlagsRepository.php +++ b/lib/Features/FeatureFlagsRepository.php @@ -7,7 +7,7 @@ use MailPoet\Entities\FeatureFlagEntity; /** * @method FeatureFlagEntity[] findAll() - * @method FeatureFlagEntity|null findOneBy(array $criteria, array $order_by = null) + * @method FeatureFlagEntity|null findOneBy(array $criteria, array $orderBy = null) * @method void persist(FeatureFlagEntity $entity) * @method void remove(FeatureFlagEntity $entity) */ diff --git a/lib/Form/Util/Styles.php b/lib/Form/Util/Styles.php index 4cfdb586f2..3094e4baa7 100644 --- a/lib/Form/Util/Styles.php +++ b/lib/Form/Util/Styles.php @@ -85,7 +85,7 @@ EOL; private $featuresController; /** - * @param FeaturesController $features_controller + * @param FeaturesController $featuresController */ public function __construct( FeaturesController $featuresController = null diff --git a/lib/Logging/LoggerFactory.php b/lib/Logging/LoggerFactory.php index 868ea8f871..c0834042ab 100644 --- a/lib/Logging/LoggerFactory.php +++ b/lib/Logging/LoggerFactory.php @@ -42,7 +42,7 @@ class LoggerFactory { /** * @param string $name - * @param bool $attach_processors + * @param bool $attachProcessors * * @return \MailPoetVendor\Monolog\Logger */ diff --git a/lib/Mailer/MailerError.php b/lib/Mailer/MailerError.php index 9bbe1fc7cb..539b05e9c3 100644 --- a/lib/Mailer/MailerError.php +++ b/lib/Mailer/MailerError.php @@ -33,8 +33,8 @@ class MailerError { * @param string $operation * @param string $level * @param null|string $message - * @param int|null $retry_interval - * @param array $subscribers_errors + * @param int|null $retryInterval + * @param array $subscribersErrors */ public function __construct($operation, $level, $message = null, $retryInterval = null, array $subscribersErrors = []) { $this->operation = $operation; diff --git a/lib/Mailer/MailerLog.php b/lib/Mailer/MailerLog.php index 4286bad4c0..d82e25bfc1 100644 --- a/lib/Mailer/MailerLog.php +++ b/lib/Mailer/MailerLog.php @@ -81,8 +81,8 @@ class MailerLog { * Process error, doesn't increase retry_attempt so it will not block sending * * @param string $operation - * @param string $error_message - * @param int $retry_interval + * @param string $errorMessage + * @param int $retryInterval * * @throws \Exception */ @@ -98,9 +98,9 @@ class MailerLog { * Process error, increase retry_attempt and block sending if it goes above RETRY_INTERVAL * * @param string $operation - * @param string $error_message - * @param string $error_code - * @param bool $pause_sending + * @param string $errorMessage + * @param string $errorCode + * @param bool $pauseSending * * @throws \Exception */ diff --git a/lib/Models/DynamicSegment.php b/lib/Models/DynamicSegment.php index dc73eca931..f655465a85 100644 --- a/lib/Models/DynamicSegment.php +++ b/lib/Models/DynamicSegment.php @@ -10,8 +10,8 @@ use MailPoet\WP\Functions as WPFunctions; * @property int $id * @property string $name * @property string $description - * @property string $created_at - * @property string $updated_at + * @property string $createdAt + * @property string $updatedAt */ class DynamicSegment extends MailPoetSegment { diff --git a/lib/Models/DynamicSegmentFilter.php b/lib/Models/DynamicSegmentFilter.php index 1c4c63f3bd..bf7b151bec 100644 --- a/lib/Models/DynamicSegmentFilter.php +++ b/lib/Models/DynamicSegmentFilter.php @@ -6,8 +6,8 @@ use MailPoet\Models\Model; use MailPoet\WP\Functions as WPFunctions; /** - * @property array|string|null $filter_data - * @property string $segment_id + * @property array|string|null $filterData + * @property string $segmentId */ class DynamicSegmentFilter extends Model { diff --git a/lib/Models/Model.php b/lib/Models/Model.php index 0b2d324ff7..f00a922e62 100644 --- a/lib/Models/Model.php +++ b/lib/Models/Model.php @@ -122,8 +122,8 @@ use MailPoet\WP\Functions as WPFunctions; * @method static static distinct() * @method $this set(string|array $key, string|null $value = null) * - * @property string|null $created_at - * @property string|null $updated_at + * @property string|null $createdAt + * @property string|null $updatedAt * @property string|null $id * @property string|null $first * @property string|null $last diff --git a/lib/Models/Newsletter.php b/lib/Models/Newsletter.php index f23d62f6d0..98d14370ca 100644 --- a/lib/Models/Newsletter.php +++ b/lib/Models/Newsletter.php @@ -20,20 +20,20 @@ use function MailPoetVendor\array_column; /** * @property int $id - * @property int $parent_id + * @property int $parentId * @property string $type * @property object|array|boolean $queue * @property string $hash - * @property string $sender_address + * @property string $senderAddress * @property string $status * @property string|object $meta * @property array $options - * @property int $children_count + * @property int $childrenCount * @property bool|array $statistics - * @property string $sent_at - * @property string $deleted_at - * @property int $total_sent - * @property int $total_scheduled + * @property string $sentAt + * @property string $deletedAt + * @property int $totalSent + * @property int $totalScheduled * @property array $segments * @property string $subject * @property string $preheader @@ -41,7 +41,7 @@ use function MailPoetVendor\array_column; * @property string|null $schedule * @property boolean|null $isScheduled * @property string|null $scheduledAt - * @property string $ga_campaign + * @property string $gaCampaign */ class Newsletter extends Model { diff --git a/lib/Models/NewsletterLink.php b/lib/Models/NewsletterLink.php index a26d6eaf27..18f7fec25d 100644 --- a/lib/Models/NewsletterLink.php +++ b/lib/Models/NewsletterLink.php @@ -3,8 +3,8 @@ namespace MailPoet\Models; /** - * @property int $newsletter_id - * @property int $queue_id + * @property int $newsletterId + * @property int $queueId * @property string $url * @property string $hash * @property int|null $clicksCount diff --git a/lib/Models/NewsletterOption.php b/lib/Models/NewsletterOption.php index 93689cbe23..2c3d40393b 100644 --- a/lib/Models/NewsletterOption.php +++ b/lib/Models/NewsletterOption.php @@ -3,10 +3,10 @@ namespace MailPoet\Models; /** - * @property int $newsletter_id - * @property int $option_field_id + * @property int $newsletterId + * @property int $optionFieldId * @property string $value - * @property string $updated_at + * @property string $updatedAt */ class NewsletterOption extends Model { public static $_table = MP_NEWSLETTER_OPTION_TABLE; diff --git a/lib/Models/NewsletterOptionField.php b/lib/Models/NewsletterOptionField.php index 4abeb73a69..f9628e9332 100644 --- a/lib/Models/NewsletterOptionField.php +++ b/lib/Models/NewsletterOptionField.php @@ -6,7 +6,7 @@ use MailPoet\WP\Functions as WPFunctions; /** * @property string $name - * @property string $newsletter_type + * @property string $newsletterType */ class NewsletterOptionField extends Model { diff --git a/lib/Models/NewsletterPost.php b/lib/Models/NewsletterPost.php index 98f922472d..4740afe46e 100644 --- a/lib/Models/NewsletterPost.php +++ b/lib/Models/NewsletterPost.php @@ -3,9 +3,9 @@ namespace MailPoet\Models; /** - * @property int $newsletter_id - * @property int $post_id - * @property string $updated_at + * @property int $newsletterId + * @property int $postId + * @property string $updatedAt */ class NewsletterPost extends Model { public static $_table = MP_NEWSLETTER_POSTS_TABLE; diff --git a/lib/Models/NewsletterSegment.php b/lib/Models/NewsletterSegment.php index 1db6560741..5237baeb6d 100644 --- a/lib/Models/NewsletterSegment.php +++ b/lib/Models/NewsletterSegment.php @@ -3,9 +3,9 @@ namespace MailPoet\Models; /** - * @property int $newsletter_id - * @property int $segment_id - * @property string $updated_at + * @property int $newsletterId + * @property int $segmentId + * @property string $updatedAt */ class NewsletterSegment extends Model { public static $_table = MP_NEWSLETTER_SEGMENT_TABLE; diff --git a/lib/Models/NewsletterTemplate.php b/lib/Models/NewsletterTemplate.php index 8757b714de..a300e22d3d 100644 --- a/lib/Models/NewsletterTemplate.php +++ b/lib/Models/NewsletterTemplate.php @@ -6,7 +6,7 @@ use MailPoet\WP\Functions as WPFunctions; /** * @property string $name - * @property int|null $newsletter_id + * @property int|null $newsletterId * @property string $categories * @property string $description * @property string|null $body diff --git a/lib/Models/ScheduledTask.php b/lib/Models/ScheduledTask.php index e27f1b29e7..d603465415 100644 --- a/lib/Models/ScheduledTask.php +++ b/lib/Models/ScheduledTask.php @@ -10,13 +10,13 @@ use MailPoetVendor\Idiorm\ORM; /** * @property int $id - * @property string $processed_at + * @property string $processedAt * @property string|null $status * @property string|null $type * @property int $priority - * @property string|null $scheduled_at - * @property boolean|null $in_progress - * @property int $reschedule_count + * @property string|null $scheduledAt + * @property boolean|null $inProgress + * @property int $rescheduleCount * @property string|array|null $meta */ class ScheduledTask extends Model { diff --git a/lib/Models/ScheduledTaskSubscriber.php b/lib/Models/ScheduledTaskSubscriber.php index 7d8c56bd1a..bdf6b2936a 100644 --- a/lib/Models/ScheduledTaskSubscriber.php +++ b/lib/Models/ScheduledTaskSubscriber.php @@ -5,8 +5,8 @@ namespace MailPoet\Models; use MailPoet\WP\Functions as WPFunctions; /** - * @property int $task_id - * @property int $subscriber_id + * @property int $taskId + * @property int $subscriberId * @property int $processed * @property int $failed * @property string $error diff --git a/lib/Models/Segment.php b/lib/Models/Segment.php index 12061922e2..b500b7d597 100644 --- a/lib/Models/Segment.php +++ b/lib/Models/Segment.php @@ -8,8 +8,8 @@ use MailPoet\WooCommerce\Helper as WCHelper; use MailPoet\WP\Functions as WPFunctions; /** - * @property array $subscribers_count - * @property array $automated_emails_subjects + * @property array $subscribersCount + * @property array $automatedEmailsSubjects * @property string $name * @property string $type * @property string $description diff --git a/lib/Models/SendingQueue.php b/lib/Models/SendingQueue.php index 49f4d7001c..1f69ae2100 100644 --- a/lib/Models/SendingQueue.php +++ b/lib/Models/SendingQueue.php @@ -8,17 +8,17 @@ use MailPoet\Util\Helpers; use MailPoet\WP\Functions as WPFunctions; /** - * @property int $count_processed - * @property int $count_to_process - * @property int $count_total - * @property string|array $newsletter_rendered_body - * @property string $newsletter_rendered_subject - * @property int $task_id - * @property int $newsletter_id + * @property int $countProcessed + * @property int $countToProcess + * @property int $countTotal + * @property string|array $newsletterRenderedBody + * @property string $newsletterRenderedSubject + * @property int $taskId + * @property int $newsletterId * @property string|object|null $meta * @property string|array $subscribers - * @property string|null $deleted_at - * @property string $scheduled_at + * @property string|null $deletedAt + * @property string $scheduledAt * @property string $status */ diff --git a/lib/Models/StatisticsClicks.php b/lib/Models/StatisticsClicks.php index 42ce0b8286..85382bd097 100644 --- a/lib/Models/StatisticsClicks.php +++ b/lib/Models/StatisticsClicks.php @@ -5,10 +5,10 @@ namespace MailPoet\Models; use DateTimeInterface; /** - * @property int $newsletter_id - * @property int $subscriber_id - * @property int $queue_id - * @property int $link_id + * @property int $newsletterId + * @property int $subscriberId + * @property int $queueId + * @property int $linkId * @property int $count */ class StatisticsClicks extends Model { diff --git a/lib/Models/StatisticsNewsletters.php b/lib/Models/StatisticsNewsletters.php index 790e8dc733..8fb0206c5b 100644 --- a/lib/Models/StatisticsNewsletters.php +++ b/lib/Models/StatisticsNewsletters.php @@ -3,7 +3,7 @@ namespace MailPoet\Models; /** - * @property string|null $sent_at + * @property string|null $sentAt */ class StatisticsNewsletters extends Model { public static $_table = MP_STATISTICS_NEWSLETTERS_TABLE; diff --git a/lib/Models/StatisticsOpens.php b/lib/Models/StatisticsOpens.php index a750bd2023..43a0d98283 100644 --- a/lib/Models/StatisticsOpens.php +++ b/lib/Models/StatisticsOpens.php @@ -3,9 +3,9 @@ namespace MailPoet\Models; /** - * @property int $newsletter_id - * @property int $subscriber_id - * @property int $queue_id + * @property int $newsletterId + * @property int $subscriberId + * @property int $queueId */ class StatisticsOpens extends Model { public static $_table = MP_STATISTICS_OPENS_TABLE; diff --git a/lib/Models/StatisticsUnsubscribes.php b/lib/Models/StatisticsUnsubscribes.php index 2a6a4f96dd..09a9fb8e37 100644 --- a/lib/Models/StatisticsUnsubscribes.php +++ b/lib/Models/StatisticsUnsubscribes.php @@ -3,9 +3,9 @@ namespace MailPoet\Models; /** - * @property int $newsletter_id - * @property int $subscriber_id - * @property int $queue_id + * @property int $newsletterId + * @property int $subscriberId + * @property int $queueId */ class StatisticsUnsubscribes extends Model { public static $_table = MP_STATISTICS_UNSUBSCRIBES_TABLE; diff --git a/lib/Models/StatisticsWooCommercePurchases.php b/lib/Models/StatisticsWooCommercePurchases.php index c6020b6ce0..4fadee2653 100644 --- a/lib/Models/StatisticsWooCommercePurchases.php +++ b/lib/Models/StatisticsWooCommercePurchases.php @@ -5,13 +5,13 @@ namespace MailPoet\Models; use WC_Order; /** - * @property int $newsletter_id - * @property int $subscriber_id - * @property int $queue_id - * @property int $click_id - * @property int $order_id - * @property string $order_currency - * @property float $order_price_total + * @property int $newsletterId + * @property int $subscriberId + * @property int $queueId + * @property int $clickId + * @property int $orderId + * @property string $orderCurrency + * @property float $orderPriceTotal */ class StatisticsWooCommercePurchases extends Model { public static $_table = MP_STATISTICS_WOOCOMMERCE_PURCHASES_TABLE; diff --git a/lib/Models/Subscriber.php b/lib/Models/Subscriber.php index 4c87aa481d..bf2b12c3c7 100644 --- a/lib/Models/Subscriber.php +++ b/lib/Models/Subscriber.php @@ -13,22 +13,22 @@ use function MailPoetVendor\array_column; /** * @property int $id * @property string $email - * @property string $first_name - * @property string $last_name + * @property string $firstName + * @property string $lastName * @property string $status - * @property string|null $subscribed_ip - * @property string|null $confirmed_ip - * @property string|null $confirmed_at - * @property string|null $last_subscribed_at - * @property string|null $deleted_at + * @property string|null $subscribedIp + * @property string|null $confirmedIp + * @property string|null $confirmedAt + * @property string|null $lastSubscribedAt + * @property string|null $deletedAt * @property string|null $source - * @property string|null $link_token - * @property int $count_confirmations - * @property int $wp_user_id + * @property string|null $linkToken + * @property int $countConfirmations + * @property int $wpUserId * @property array $segments * @property array $subscriptions - * @property string $unconfirmed_data - * @property int $is_woocommerce_user + * @property string $unconfirmedData + * @property int $isWoocommerceUser */ class Subscriber extends Model { @@ -346,7 +346,7 @@ class Subscriber extends Model { } /** - * @param string $customer_email + * @param string $customerEmail * @return bool|Subscriber */ public static function getWooCommerceSegmentSubscriber($customerEmail) { diff --git a/lib/Models/SubscriberCustomField.php b/lib/Models/SubscriberCustomField.php index 1b01efdf2b..e695a67e1b 100644 --- a/lib/Models/SubscriberCustomField.php +++ b/lib/Models/SubscriberCustomField.php @@ -7,8 +7,8 @@ use MailPoet\Util\Helpers; use function MailPoetVendor\array_column; /** - * @property int $subscriber_id - * @property int $custom_field_id + * @property int $subscriberId + * @property int $customFieldId * @property string $value */ class SubscriberCustomField extends Model { diff --git a/lib/Models/SubscriberSegment.php b/lib/Models/SubscriberSegment.php index 1e62f65b50..5bcbf8d554 100644 --- a/lib/Models/SubscriberSegment.php +++ b/lib/Models/SubscriberSegment.php @@ -4,8 +4,8 @@ namespace MailPoet\Models; /** * @property int $id - * @property int $subscriber_id - * @property int $segment_id + * @property int $subscriberId + * @property int $segmentId * @property string $status */ diff --git a/lib/Newsletter/NewslettersRepository.php b/lib/Newsletter/NewslettersRepository.php index ce86ee5f0a..2a125bfb87 100644 --- a/lib/Newsletter/NewslettersRepository.php +++ b/lib/Newsletter/NewslettersRepository.php @@ -6,8 +6,8 @@ use MailPoet\Doctrine\Repository; use MailPoet\Entities\NewsletterEntity; /** - * @method NewsletterEntity[] findBy(array $criteria, array $order_by = null, int $limit = null, int $offset = null) - * @method NewsletterEntity|null findOneBy(array $criteria, array $order_by = null) + * @method NewsletterEntity[] findBy(array $criteria, array $orderBy = null, int $limit = null, int $offset = null) + * @method NewsletterEntity|null findOneBy(array $criteria, array $orderBy = null) * @method NewsletterEntity|null findOneById(mixed $id) * @method void persist(NewsletterEntity $entity) * @method void remove(NewsletterEntity $entity) diff --git a/lib/Newsletter/Renderer/Renderer.php b/lib/Newsletter/Renderer/Renderer.php index cacde4584e..97783a97ba 100644 --- a/lib/Newsletter/Renderer/Renderer.php +++ b/lib/Newsletter/Renderer/Renderer.php @@ -162,7 +162,7 @@ class Renderer { } /** - * @param DomNode $template_dom + * @param DomNode $templateDom * @return string */ private function postProcessTemplate(DomNode $templateDom) { diff --git a/lib/Settings/SettingsRepository.php b/lib/Settings/SettingsRepository.php index e8f454adb4..29b8fc02cc 100644 --- a/lib/Settings/SettingsRepository.php +++ b/lib/Settings/SettingsRepository.php @@ -8,8 +8,8 @@ use MailPoet\WP\Functions as WPFunctions; use MailPoetVendor\Carbon\Carbon; /** - * @method SettingEntity[] findBy(array $criteria, array $order_by = null, int $limit = null, int $offset = null) - * @method SettingEntity|null findOneBy(array $criteria, array $order_by = null) + * @method SettingEntity[] findBy(array $criteria, array $orderBy = null, int $limit = null, int $offset = null) + * @method SettingEntity|null findOneBy(array $criteria, array $orderBy = null) * @method SettingEntity|null findOneById(mixed $id) * @method SettingEntity[] findAll() * @method void persist(SettingEntity $entity) diff --git a/lib/Settings/UserFlagsRepository.php b/lib/Settings/UserFlagsRepository.php index 684ee5cd59..bae0dc374c 100644 --- a/lib/Settings/UserFlagsRepository.php +++ b/lib/Settings/UserFlagsRepository.php @@ -6,8 +6,8 @@ use MailPoet\Doctrine\Repository; use MailPoet\Entities\UserFlagEntity; /** - * @method UserFlagEntity[] findBy(array $criteria, array $order_by = null, int $limit = null, int $offset = null) - * @method UserFlagEntity|null findOneBy(array $criteria, array $order_by = null) + * @method UserFlagEntity[] findBy(array $criteria, array $orderBy = null, int $limit = null, int $offset = null) + * @method UserFlagEntity|null findOneBy(array $criteria, array $orderBy = null) * @method UserFlagEntity|null findOneById(mixed $id) * @method void persist(UserFlagEntity $entity) * @method void remove(UserFlagEntity $entity) diff --git a/lib/Subscribers/InactiveSubscribersController.php b/lib/Subscribers/InactiveSubscribersController.php index 6e00647f4b..2bf5315133 100644 --- a/lib/Subscribers/InactiveSubscribersController.php +++ b/lib/Subscribers/InactiveSubscribersController.php @@ -24,8 +24,8 @@ class InactiveSubscribersController { } /** - * @param int $days_to_inactive - * @param int $batch_size + * @param int $daysToInactive + * @param int $batchSize * @return int|boolean */ public function markInactiveSubscribers($daysToInactive, $batchSize, $startId = null) { @@ -34,8 +34,8 @@ class InactiveSubscribersController { } /** - * @param int $days_to_inactive - * @param int $batch_size + * @param int $daysToInactive + * @param int $batchSize * @return int */ public function markActiveSubscribers($daysToInactive, $batchSize) { @@ -55,7 +55,7 @@ class InactiveSubscribersController { } /** - * @param int $days_to_inactive + * @param int $daysToInactive * @return Carbon */ private function getThresholdDate($daysToInactive) { @@ -64,8 +64,8 @@ class InactiveSubscribersController { } /** - * @param Carbon $threshold_date - * @param int $batch_size + * @param Carbon $thresholdDate + * @param int $batchSize * @return int|boolean */ private function deactivateSubscribers(Carbon $thresholdDate, $batchSize, $startId = null) { @@ -149,8 +149,8 @@ class InactiveSubscribersController { } /** - * @param Carbon $threshold_date - * @param int $batch_size + * @param Carbon $thresholdDate + * @param int $batchSize * @return int */ private function activateSubscribers(Carbon $thresholdDate, $batchSize) { diff --git a/lib/Subscribers/SubscribersRepository.php b/lib/Subscribers/SubscribersRepository.php index 8dc15f7f90..eb17d8cb61 100644 --- a/lib/Subscribers/SubscribersRepository.php +++ b/lib/Subscribers/SubscribersRepository.php @@ -6,8 +6,8 @@ use MailPoet\Doctrine\Repository; use MailPoet\Entities\SubscriberEntity; /** - * @method SubscriberEntity[] findBy(array $criteria, array $order_by = null, int $limit = null, int $offset = null) - * @method SubscriberEntity|null findOneBy(array $criteria, array $order_by = null) + * @method SubscriberEntity[] findBy(array $criteria, array $orderBy = null, int $limit = null, int $offset = null) + * @method SubscriberEntity|null findOneBy(array $criteria, array $orderBy = null) * @method SubscriberEntity|null findOneById(mixed $id) * @method void persist(SubscriberEntity $entity) * @method void remove(SubscriberEntity $entity) @@ -34,4 +34,4 @@ class SubscribersRepository extends Repository { ->getQuery(); return (int)$query->getSingleScalarResult(); } -} \ No newline at end of file +} diff --git a/lib/Tasks/Sending.php b/lib/Tasks/Sending.php index 74d5bf5fe4..49860b5a43 100644 --- a/lib/Tasks/Sending.php +++ b/lib/Tasks/Sending.php @@ -13,12 +13,12 @@ use function MailPoetVendor\array_column; /** * A facade class containing all necessary models to work with a sending queue * @property string|null $status - * @property int $task_id + * @property int $taskId * @property int $id - * @property int $newsletter_id - * @property string $newsletter_rendered_subject - * @property string|array $newsletter_rendered_body - * @property bool $non_existent_column + * @property int $newsletterId + * @property string $newsletterRenderedSubject + * @property string|array $newsletterRenderedBody + * @property bool $nonExistentColumn */ class Sending { const TASK_TYPE = 'sending'; diff --git a/lib/WP/Functions.php b/lib/WP/Functions.php index 9dfbf1c10b..d201de472f 100644 --- a/lib/WP/Functions.php +++ b/lib/WP/Functions.php @@ -42,9 +42,9 @@ class Functions { /** * @param string $tag - * @param callable $function_to_add + * @param callable $functionToAdd * @param int $priority - * @param int $accepted_args + * @param int $acceptedArgs * @return boolean */ public function addAction($tag, $functionToAdd, $priority = 10, $acceptedArgs = 1) { @@ -260,14 +260,14 @@ class Functions { } /** - * @param int|false $user_id + * @param int|false $userId */ public function getTheAuthorMeta($field = '', $userId = false) { return get_the_author_meta($field, $userId); } /** - * @param int|\WP_User $user_id + * @param int|\WP_User $userId */ public function getUserLocale($userId = 0) { return get_user_locale($userId); @@ -327,7 +327,7 @@ class Functions { /** * @param string|false $deprecated - * @param string|false $plugin_rel_path + * @param string|false $pluginRelPath */ public function loadPluginTextdomain($domain, $deprecated = false, $pluginRelPath = false) { return load_plugin_textdomain($domain, $deprecated, $pluginRelPath); @@ -363,7 +363,7 @@ class Functions { /** * @param string $tag - * @param callable $function_to_remove + * @param callable $functionToRemove * @param int $priority */ public function removeAction($tag, $functionToRemove, $priority = 10) { diff --git a/tests/DataFactories/Newsletter.php b/tests/DataFactories/Newsletter.php index f921f14f6f..6357091f7f 100644 --- a/tests/DataFactories/Newsletter.php +++ b/tests/DataFactories/Newsletter.php @@ -73,7 +73,7 @@ class Newsletter { } /** - * @param string $created_at in format Y-m-d H:i:s + * @param string $createdAt in format Y-m-d H:i:s * @return Newsletter */ public function withCreatedAt($createdAt) { diff --git a/tests/DataFactories/NewsletterLink.php b/tests/DataFactories/NewsletterLink.php index 54ecdf94af..9b7bc58aee 100644 --- a/tests/DataFactories/NewsletterLink.php +++ b/tests/DataFactories/NewsletterLink.php @@ -28,7 +28,7 @@ class NewsletterLink { } /** - * @param string $created_at in format Y-m-d H:i:s + * @param string $createdAt in format Y-m-d H:i:s * @return NewsletterLink */ public function withCreatedAt($createdAt) { diff --git a/tests/DataFactories/Subscriber.php b/tests/DataFactories/Subscriber.php index 8b5a4d15e7..e7aff98cf3 100644 --- a/tests/DataFactories/Subscriber.php +++ b/tests/DataFactories/Subscriber.php @@ -21,7 +21,7 @@ class Subscriber { } /** - * @param string $first_name + * @param string $firstName * @return $this */ public function withFirstName($firstName) { @@ -30,7 +30,7 @@ class Subscriber { } /** - * @param string $last_name + * @param string $lastName * @return $this */ public function withLastName($lastName) { diff --git a/tests/DataFactories/WooCommerceOrder.php b/tests/DataFactories/WooCommerceOrder.php index af2de7cdf8..2680ed1321 100644 --- a/tests/DataFactories/WooCommerceOrder.php +++ b/tests/DataFactories/WooCommerceOrder.php @@ -47,7 +47,7 @@ class WooCommerceOrder { } /** - * @param array $customer_data Customer created via WooCommerceCustomer factory + * @param array $customerData Customer created via WooCommerceCustomer factory * @return $this */ public function withCustomer($customerData) { diff --git a/tests/acceptance/RevenueTrackingCookieCest.php b/tests/acceptance/RevenueTrackingCookieCest.php index 02b5b18eca..2c7ae0d2e6 100644 --- a/tests/acceptance/RevenueTrackingCookieCest.php +++ b/tests/acceptance/RevenueTrackingCookieCest.php @@ -100,7 +100,7 @@ class RevenueTrackingCookieCest { * Checks that email was received by looking for a subject in inbox. * In case it was not found reloads the inbox and check once more. * Emails are sent via cron and might not be sent immediately. - * @param \AcceptanceTester $I + * @param \AcceptanceTester $i * @param string $subject * @throws \Exception */ diff --git a/tests/acceptance/WooCommerceListImportPageCest.php b/tests/acceptance/WooCommerceListImportPageCest.php index 351d3a1aac..612472d32f 100644 --- a/tests/acceptance/WooCommerceListImportPageCest.php +++ b/tests/acceptance/WooCommerceListImportPageCest.php @@ -87,7 +87,7 @@ class WooCommerceListImportPageCest { /** * Test that admin is always redirected to WooCommerce list import page and * can't go to another page unless he submits the form - * @param \AcceptanceTester $I + * @param \AcceptanceTester $i */ public function importListPageRedirectionTest(\AcceptanceTester $i) { $this->settings->withWooCommerceListImportPageDisplayed(false); diff --git a/tests/integration/API/JSON/v1/NewslettersTest.php b/tests/integration/API/JSON/v1/NewslettersTest.php index b004c35ce3..052b7488b0 100644 --- a/tests/integration/API/JSON/v1/NewslettersTest.php +++ b/tests/integration/API/JSON/v1/NewslettersTest.php @@ -391,10 +391,10 @@ class NewslettersTest extends \MailPoetTest { ], ]; $newsletter = $this->endpoint->save($newsletterData); - /** @var SendingQueue $sending_queue_1 */ + /** @var SendingQueue $sendingQueue1 */ $sendingQueue1 = SendingQueue::findOne($sendingQueue1->id); $sendingQueue1 = SendingTask::createFromQueue($sendingQueue1); - /** @var SendingQueue $sending_queue_2 */ + /** @var SendingQueue $sendingQueue2 */ $sendingQueue2 = SendingQueue::findOne($sendingQueue2->id); $sendingQueue2 = SendingTask::createFromQueue($sendingQueue2); expect($sendingQueue1->scheduledAt)->notEquals($currentTime); diff --git a/tests/integration/Cron/Workers/SchedulerTest.php b/tests/integration/Cron/Workers/SchedulerTest.php index dae893cd89..734dcb735a 100644 --- a/tests/integration/Cron/Workers/SchedulerTest.php +++ b/tests/integration/Cron/Workers/SchedulerTest.php @@ -261,7 +261,7 @@ class SchedulerTest extends \MailPoetTest { ], $this); expect($queue->status)->notNull(); expect($scheduler->processWelcomeNewsletter($newsletter, $queue))->true(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect($updatedQueue->status)->null(); @@ -280,7 +280,7 @@ class SchedulerTest extends \MailPoetTest { expect($queue->status)->notNull(); expect($scheduler->processWelcomeNewsletter($newsletter, $queue))->true(); // update queue's status to null - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect($updatedQueue->status)->null(); @@ -338,7 +338,7 @@ class SchedulerTest extends \MailPoetTest { $result = $scheduler->verifyMailpoetSubscriber($subscriber->id, $newsletter, $queue); expect($result)->false(); // update the time queue is scheduled to run at - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect(Carbon::parse($updatedQueue->scheduledAt))->equals( @@ -413,7 +413,7 @@ class SchedulerTest extends \MailPoetTest { // return true expect($scheduler->processScheduledStandardNewsletter($newsletter, $queue))->true(); // update queue's list of subscribers to process - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); $updatedQueueSubscribers = $updatedQueue->getSubscribers(ScheduledTaskSubscriber::STATUS_UNPROCESSED); @@ -480,7 +480,7 @@ class SchedulerTest extends \MailPoetTest { expect($notificationHistory)->notEmpty(); // update queue with a list of subscribers to process and change newsletter id // to that of the notification history - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); $updatedQueueSubscribers = $updatedQueue->getSubscribers(ScheduledTaskSubscriber::STATUS_UNPROCESSED); diff --git a/tests/integration/Cron/Workers/SendingQueue/SendingQueueTest.php b/tests/integration/Cron/Workers/SendingQueue/SendingQueueTest.php index b78877ee87..5421ce6faa 100644 --- a/tests/integration/Cron/Workers/SendingQueue/SendingQueueTest.php +++ b/tests/integration/Cron/Workers/SendingQueue/SendingQueueTest.php @@ -357,7 +357,7 @@ class SendingQueueTest extends \MailPoetTest { expect($updatedNewsletter->status)->equals(Newsletter::STATUS_SENT); // queue status is set to completed - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect($updatedQueue->status)->equals(SendingQueue::STATUS_COMPLETED); @@ -409,7 +409,7 @@ class SendingQueueTest extends \MailPoetTest { expect($updatedNewsletter->status)->equals(Newsletter::STATUS_SENT); // queue status is set to completed - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect($updatedQueue->status)->equals(SendingQueue::STATUS_COMPLETED); @@ -454,7 +454,7 @@ class SendingQueueTest extends \MailPoetTest { $sendingQueueWorker->process(); // queue status is set to completed - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect($updatedQueue->status)->equals(SendingQueue::STATUS_COMPLETED); @@ -535,7 +535,7 @@ class SendingQueueTest extends \MailPoetTest { expect($updatedNewsletter->status)->equals(Newsletter::STATUS_SENT); // queue status is set to completed - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect($updatedQueue->status)->equals(SendingQueue::STATUS_COMPLETED); @@ -580,7 +580,7 @@ class SendingQueueTest extends \MailPoetTest { $sendingQueueWorker->process(); // queue status is set to completed - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); @@ -611,7 +611,7 @@ class SendingQueueTest extends \MailPoetTest { ); $sendingQueueWorker->process(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); // queue subscriber processed/to process count is updated @@ -643,7 +643,7 @@ class SendingQueueTest extends \MailPoetTest { ); $sendingQueueWorker->process(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); // queue subscriber processed/to process count is updated @@ -665,7 +665,7 @@ class SendingQueueTest extends \MailPoetTest { // newsletter is sent to existing subscriber $sendingQueueWorker->process(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect((int)$updatedQueue->countTotal)->equals(1); @@ -677,7 +677,7 @@ class SendingQueueTest extends \MailPoetTest { $subscriber->deletedAt = Carbon::now(); $subscriber->save(); $sendingQueueWorker->process(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect((int)$updatedQueue->countTotal)->equals(0); @@ -695,7 +695,7 @@ class SendingQueueTest extends \MailPoetTest { $subscriber->status = Subscriber::STATUS_UNSUBSCRIBED; $subscriber->save(); $sendingQueueWorker->process(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect((int)$updatedQueue->countTotal)->equals(0); @@ -713,7 +713,7 @@ class SendingQueueTest extends \MailPoetTest { $subscriberSegment->status = Subscriber::STATUS_UNSUBSCRIBED; $subscriberSegment->save(); $sendingQueueWorker->process(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect((int)$updatedQueue->countTotal)->equals(0); @@ -731,7 +731,7 @@ class SendingQueueTest extends \MailPoetTest { $subscriber->status = Subscriber::STATUS_INACTIVE; $subscriber->save(); $sendingQueueWorker->process(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); expect((int)$updatedQueue->countTotal)->equals(0); diff --git a/tests/integration/Cron/Workers/SendingQueue/Tasks/NewsletterTest.php b/tests/integration/Cron/Workers/SendingQueue/Tasks/NewsletterTest.php index 3555d83cc3..1a7ee05897 100644 --- a/tests/integration/Cron/Workers/SendingQueue/Tasks/NewsletterTest.php +++ b/tests/integration/Cron/Workers/SendingQueue/Tasks/NewsletterTest.php @@ -159,7 +159,7 @@ class NewsletterTest extends \MailPoetTest { $newsletterTask->preProcessNewsletter($this->newsletter, $this->queue); $link = NewsletterLink::where('newsletter_id', $this->newsletter->id) ->findOne(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); $renderedNewsletter = $updatedQueue->getNewsletterRenderedBody(); @@ -184,7 +184,7 @@ class NewsletterTest extends \MailPoetTest { $link = NewsletterLink::where('newsletter_id', $this->newsletter->id) ->findOne(); expect($link)->false(); - /** @var SendingQueue $updated_queue */ + /** @var SendingQueue $updatedQueue */ $updatedQueue = SendingQueue::findOne($this->queue->id); $updatedQueue = SendingTask::createFromQueue($updatedQueue); $renderedNewsletter = $updatedQueue->getNewsletterRenderedBody(); diff --git a/tests/integration/Cron/Workers/SimpleWorkerTest.php b/tests/integration/Cron/Workers/SimpleWorkerTest.php index 60f52550fc..c9e54c010d 100644 --- a/tests/integration/Cron/Workers/SimpleWorkerTest.php +++ b/tests/integration/Cron/Workers/SimpleWorkerTest.php @@ -55,7 +55,7 @@ class SimpleWorkerTest extends \MailPoetTest { public function testItCalculatesNextRunDateWithinNextWeekBoundaries() { $currentDate = Carbon::createFromTimestamp(WPFunctions::get()->currentTime('timestamp')); - /** @var Carbon $next_run_date */ + /** @var Carbon $nextRunDate */ $nextRunDate = (new MockSimpleWorker())->getNextRunDate(); $difference = $nextRunDate->diffInDays($currentDate); // Subtract days left in the current week diff --git a/tests/integration/Doctrine/Types/JsonEntity.php b/tests/integration/Doctrine/Types/JsonEntity.php index de31abbd76..b33e68d47e 100644 --- a/tests/integration/Doctrine/Types/JsonEntity.php +++ b/tests/integration/Doctrine/Types/JsonEntity.php @@ -44,7 +44,7 @@ class JsonEntity { } /** - * @param array|null $json_data + * @param array|null $jsonData */ public function setJsonData($jsonData) { $this->jsonData = $jsonData; @@ -58,7 +58,7 @@ class JsonEntity { } /** - * @param array|null $json_or_serialized_data + * @param array|null $jsonOrSerializedData */ public function setJsonOrSerializedData($jsonOrSerializedData) { $this->jsonOrSerializedData = $jsonOrSerializedData; diff --git a/tests/integration/DynamicSegments/FreePluginConnectors/AddToNewslettersSegmentsTest.php b/tests/integration/DynamicSegments/FreePluginConnectors/AddToNewslettersSegmentsTest.php index 91c39a9ac1..fad80b53e4 100644 --- a/tests/integration/DynamicSegments/FreePluginConnectors/AddToNewslettersSegmentsTest.php +++ b/tests/integration/DynamicSegments/FreePluginConnectors/AddToNewslettersSegmentsTest.php @@ -39,7 +39,7 @@ class AddToNewslettersSegmentsTest extends \MailPoetTest { return [$dynamicSegment]; })]); - /** @var \MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount|MockObject $subscribers_count_loader */ + /** @var \MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount|MockObject $subscribersCountLoader */ $subscribersCountLoader = Stub::makeEmpty('\MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount', ['getSubscribersCount']); $subscribersCountLoader ->expects($this->once()) diff --git a/tests/integration/DynamicSegments/FreePluginConnectors/AddToSubscribersFiltersTest.php b/tests/integration/DynamicSegments/FreePluginConnectors/AddToSubscribersFiltersTest.php index fc611129a8..de458d3171 100644 --- a/tests/integration/DynamicSegments/FreePluginConnectors/AddToSubscribersFiltersTest.php +++ b/tests/integration/DynamicSegments/FreePluginConnectors/AddToSubscribersFiltersTest.php @@ -38,7 +38,7 @@ class AddToSubscribersFiltersTest extends \MailPoetTest { return [$dynamicSegment]; })]); - /** @var \MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount|MockObject $subscribers_count_loader */ + /** @var \MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount|MockObject $subscribersCountLoader */ $subscribersCountLoader = Stub::makeEmpty('\MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount', ['getSubscribersCount']); $subscribersCountLoader ->expects($this->once()) @@ -79,7 +79,7 @@ class AddToSubscribersFiltersTest extends \MailPoetTest { ] ); - /** @var \MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount|MockObject $subscribers_count_loader */ + /** @var \MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount|MockObject $subscribersCountLoader */ $subscribersCountLoader = Stub::makeEmpty('\MailPoet\DynamicSegments\Persistence\Loading\SubscribersCount', ['getSubscribersCount']); $subscribersCountLoader ->expects($this->exactly(2)) diff --git a/tests/integration/Models/ScheduledTaskTest.php b/tests/integration/Models/ScheduledTaskTest.php index 3a979293e2..e3a1f3ebb3 100644 --- a/tests/integration/Models/ScheduledTaskTest.php +++ b/tests/integration/Models/ScheduledTaskTest.php @@ -125,7 +125,7 @@ class ScheduledTaskTest extends \MailPoetTest { $task = ScheduledTask::findOne($task->id); - /** @var string $task_meta */ + /** @var string $taskMeta */ $taskMeta = $task->meta; expect(Helpers::isJson($taskMeta))->true(); expect(json_decode($taskMeta, true))->equals($meta); diff --git a/tests/integration/Subscribers/InactiveSubscribersControllerTest.php b/tests/integration/Subscribers/InactiveSubscribersControllerTest.php index c2d3b50ba2..2fa3d10d41 100644 --- a/tests/integration/Subscribers/InactiveSubscribersControllerTest.php +++ b/tests/integration/Subscribers/InactiveSubscribersControllerTest.php @@ -239,7 +239,7 @@ class InactiveSubscribersControllerTest extends \MailPoetTest { /** * @param string $email - * @param int $created_days_ago + * @param int $createdDaysAgo * @param string $status * @return Subscriber */ @@ -253,7 +253,7 @@ class InactiveSubscribersControllerTest extends \MailPoetTest { } /** - * @param int $processed_days_ago + * @param int $processedDaysAgo * @return array */ private function createCompletedSendingTask($processedDaysAgo = 0) { @@ -268,7 +268,7 @@ class InactiveSubscribersControllerTest extends \MailPoetTest { } /** - * @param int $processed_days_ago + * @param int $processedDaysAgo * @return array */ private function createCompletedSendingTaskWithOneOpen($processedDaysAgo = 0) { @@ -282,7 +282,7 @@ class InactiveSubscribersControllerTest extends \MailPoetTest { /** * @param Subscriber $subscriber * @param ScheduledTask $task - * @param int $days_ago + * @param int $daysAgo */ private function addSubcriberToTask(Subscriber $subscriber, ScheduledTask $task, $daysAgo = 0) { $createdAt = (new Carbon())->subDays($daysAgo)->toDateTimeString();