@@ -229,7 +229,7 @@ |
||
| 229 | 229 | |
| 230 | 230 | /** |
| 231 | 231 | * @param EmailRecipientsProviderArgs $args |
| 232 | - * @param object|null $object |
|
| 232 | + * @param \Oro\Bundle\EmailBundle\Entity\EmailOwnerInterface|null $object |
|
| 233 | 233 | * |
| 234 | 234 | * @return bool |
| 235 | 235 | */ |
@@ -4,9 +4,7 @@ |
||
| 4 | 4 | |
| 5 | 5 | use Doctrine\Common\EventManager; |
| 6 | 6 | use Doctrine\DBAL\Driver\Connection; |
| 7 | - |
|
| 8 | 7 | use Symfony\Component\Filesystem\Filesystem; |
| 9 | - |
|
| 10 | 8 | use Oro\Component\TestUtils\ORM\Mocks\DriverMock; |
| 11 | 9 | use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock; |
| 12 | 10 | |
@@ -143,7 +143,7 @@ discard block |
||
| 143 | 143 | * @param EmailTemplateInterface $template |
| 144 | 144 | * @param array $templateParams |
| 145 | 145 | * |
| 146 | - * @return array first element is email subject, second - message |
|
| 146 | + * @return string[] first element is email subject, second - message |
|
| 147 | 147 | */ |
| 148 | 148 | public function compileMessage(EmailTemplateInterface $template, array $templateParams = []) |
| 149 | 149 | { |
@@ -206,7 +206,7 @@ discard block |
||
| 206 | 206 | * @param string $template |
| 207 | 207 | * @param object $entity |
| 208 | 208 | * |
| 209 | - * @return EmailTemplate |
|
| 209 | + * @return string |
|
| 210 | 210 | */ |
| 211 | 211 | protected function processDefaultFilters($template, $entity) |
| 212 | 212 | { |
@@ -4,16 +4,13 @@ |
||
| 4 | 4 | |
| 5 | 5 | use Oro\Bundle\WorkflowBundle\Model\ContextAccessor; |
| 6 | 6 | use Symfony\Component\Form\Test\FormIntegrationTestCase; |
| 7 | - |
|
| 8 | 7 | use Oro\Bundle\WorkflowBundle\Form\EventListener\DefaultValuesListener; |
| 9 | 8 | use Oro\Bundle\WorkflowBundle\Form\EventListener\InitActionsListener; |
| 10 | 9 | use Oro\Bundle\WorkflowBundle\Form\EventListener\RequiredAttributesListener; |
| 11 | 10 | use Oro\Bundle\WorkflowBundle\Form\Type\WorkflowAttributesType; |
| 12 | - |
|
| 13 | 11 | use Oro\Bundle\WorkflowBundle\Entity\WorkflowItem; |
| 14 | 12 | use Oro\Bundle\WorkflowBundle\Entity\WorkflowStep; |
| 15 | 13 | use Oro\Bundle\WorkflowBundle\Entity\WorkflowDefinition; |
| 16 | - |
|
| 17 | 14 | use Oro\Bundle\WorkflowBundle\Model\Step; |
| 18 | 15 | use Oro\Bundle\WorkflowBundle\Model\Attribute; |
| 19 | 16 | use Oro\Bundle\WorkflowBundle\Model\WorkflowData; |
@@ -97,7 +97,7 @@ |
||
| 97 | 97 | } |
| 98 | 98 | |
| 99 | 99 | /** |
| 100 | - * @param object|null $owner |
|
| 100 | + * @param \Oro\Bundle\EmailBundle\Entity\EmailOwnerInterface|null $owner |
|
| 101 | 101 | * |
| 102 | 102 | * @return RecipientEntity|null |
| 103 | 103 | */ |
@@ -3,7 +3,6 @@ |
||
| 3 | 3 | namespace Oro\Component\PropertyAccess\Tests\Unit; |
| 4 | 4 | |
| 5 | 5 | use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException; |
| 6 | - |
|
| 7 | 6 | use Oro\Component\PropertyAccess\PropertyAccessor; |
| 8 | 7 | use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass; |
| 9 | 8 | use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall; |
@@ -111,8 +111,8 @@ discard block |
||
| 111 | 111 | |
| 112 | 112 | /** |
| 113 | 113 | * @param EmailHeader $email |
| 114 | - * @param $folderType |
|
| 115 | - * @param $organization |
|
| 114 | + * @param string $folderType |
|
| 115 | + * @param OrganizationInterface $organization |
|
| 116 | 116 | * |
| 117 | 117 | * @return bool |
| 118 | 118 | * @throws \Doctrine\ORM\NonUniqueResultException |
@@ -292,7 +292,7 @@ discard block |
||
| 292 | 292 | |
| 293 | 293 | /** |
| 294 | 294 | * @param EmailFolder $folder |
| 295 | - * @param array $messageIds |
|
| 295 | + * @param string[] $messageIds |
|
| 296 | 296 | * @return EmailUser[] |
| 297 | 297 | */ |
| 298 | 298 | protected function getExistingEmailUsers(EmailFolder $folder, array $messageIds) |
@@ -356,7 +356,7 @@ discard block |
||
| 356 | 356 | } |
| 357 | 357 | |
| 358 | 358 | /** |
| 359 | - * @return array |
|
| 359 | + * @return string[] |
|
| 360 | 360 | */ |
| 361 | 361 | protected function entitiesToClear() |
| 362 | 362 | { |
@@ -3,11 +3,8 @@ |
||
| 3 | 3 | namespace Oro\Bundle\EmailBundle\Sync; |
| 4 | 4 | |
| 5 | 5 | use Doctrine\ORM\EntityManager; |
| 6 | -use Doctrine\ORM\Query; |
|
| 7 | - |
|
| 8 | 6 | use Psr\Log\LoggerAwareInterface; |
| 9 | 7 | use Psr\Log\LoggerAwareTrait; |
| 10 | - |
|
| 11 | 8 | use Oro\Bundle\EmailBundle\Entity\Provider\EmailOwnerProviderStorage; |
| 12 | 9 | use Oro\Bundle\EmailBundle\Entity\Manager\EmailAddressManager; |
| 13 | 10 | use Oro\Bundle\EmailBundle\Tools\EmailAddressHelper; |
@@ -308,7 +308,7 @@ discard block |
||
| 308 | 308 | /** |
| 309 | 309 | * Creates a processor is used to synchronize emails |
| 310 | 310 | * |
| 311 | - * @param object $origin An instance of class implementing EmailOrigin entity |
|
| 311 | + * @param EmailOrigin $origin An instance of class implementing EmailOrigin entity |
|
| 312 | 312 | * @return AbstractEmailSynchronizationProcessor |
| 313 | 313 | */ |
| 314 | 314 | abstract protected function createSynchronizationProcessor($origin); |
@@ -423,7 +423,7 @@ discard block |
||
| 423 | 423 | * Finds active email origin by its id |
| 424 | 424 | * |
| 425 | 425 | * @param int $originId |
| 426 | - * @return EmailOrigin|null |
|
| 426 | + * @return EmailOrigin |
|
| 427 | 427 | */ |
| 428 | 428 | protected function findOrigin($originId) |
| 429 | 429 | { |
@@ -4,16 +4,11 @@ |
||
| 4 | 4 | |
| 5 | 5 | use Doctrine\Common\Persistence\ManagerRegistry; |
| 6 | 6 | use Doctrine\ORM\EntityManager; |
| 7 | -use Doctrine\ORM\Query; |
|
| 8 | -use Doctrine\ORM\Query\Expr; |
|
| 9 | - |
|
| 10 | 7 | use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorage; |
| 11 | 8 | use Symfony\Component\Security\Core\Authentication\Token\TokenInterface; |
| 12 | - |
|
| 13 | 9 | use Psr\Log\LoggerAwareInterface; |
| 14 | 10 | use Psr\Log\LoggerAwareTrait; |
| 15 | 11 | use Psr\Log\NullLogger; |
| 16 | - |
|
| 17 | 12 | use Oro\Bundle\EmailBundle\Entity\EmailOrigin; |
| 18 | 13 | use Oro\Bundle\EmailBundle\Exception\SyncFolderTimeoutException; |
| 19 | 14 | use Oro\Bundle\OrganizationBundle\Entity\Organization; |
@@ -3,11 +3,8 @@ |
||
| 3 | 3 | namespace Oro\Bundle\EmailBundle\Sync; |
| 4 | 4 | |
| 5 | 5 | use Doctrine\ORM\EntityManager; |
| 6 | -use Doctrine\ORM\Query; |
|
| 7 | - |
|
| 8 | 6 | use Psr\Log\LoggerAwareInterface; |
| 9 | 7 | use Psr\Log\LoggerAwareTrait; |
| 10 | - |
|
| 11 | 8 | use Oro\Bundle\EmailBundle\Entity\Provider\EmailOwnerProviderStorage; |
| 12 | 9 | use Oro\Bundle\EmailBundle\Entity\Manager\EmailAddressManager; |
| 13 | 10 | use Oro\Bundle\EmailBundle\Tools\EmailAddressHelper; |
@@ -95,9 +95,9 @@ |
||
| 95 | 95 | * @param string $subject |
| 96 | 96 | * @param string $messageId |
| 97 | 97 | * @param string $from |
| 98 | - * @param string|string[] $to |
|
| 99 | - * @param string|string[]|null $cc |
|
| 100 | - * @param string|string[]|null $bcc |
|
| 98 | + * @param string $to |
|
| 99 | + * @param string $cc |
|
| 100 | + * @param string $bcc |
|
| 101 | 101 | * |
| 102 | 102 | * @return Email |
| 103 | 103 | */ |
@@ -4,9 +4,7 @@ |
||
| 4 | 4 | |
| 5 | 5 | use Doctrine\Common\EventManager; |
| 6 | 6 | use Doctrine\DBAL\Driver\Connection; |
| 7 | - |
|
| 8 | 7 | use Symfony\Component\Filesystem\Filesystem; |
| 9 | - |
|
| 10 | 8 | use Oro\Component\TestUtils\ORM\Mocks\DriverMock; |
| 11 | 9 | use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock; |
| 12 | 10 | |
@@ -14,7 +14,7 @@ discard block |
||
| 14 | 14 | class Email extends AbstractPageEntity |
| 15 | 15 | { |
| 16 | 16 | /** |
| 17 | - * @param $subject |
|
| 17 | + * @param string $subject |
|
| 18 | 18 | * @return $this |
| 19 | 19 | */ |
| 20 | 20 | public function setSubject($subject) |
@@ -24,7 +24,7 @@ discard block |
||
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | /** |
| 27 | - * @param $sendTo |
|
| 27 | + * @param string $sendTo |
|
| 28 | 28 | * @return $this |
| 29 | 29 | */ |
| 30 | 30 | public function setTo($sendTo) |
@@ -44,12 +44,16 @@ discard block |
||
| 44 | 44 | } |
| 45 | 45 | |
| 46 | 46 | /** |
| 47 | - * @param $content string |
|
| 47 | + * @param string $content string |
|
| 48 | 48 | * @return $this |
| 49 | 49 | */ |
| 50 | 50 | public function setBody($content) |
| 51 | 51 | { |
| 52 | 52 | $this->test->waitUntil( |
| 53 | + |
|
| 54 | + /** |
|
| 55 | + * @param string $testCase |
|
| 56 | + */ |
|
| 53 | 57 | function (\PHPUnit_Extensions_Selenium2TestCase $testCase) { |
| 54 | 58 | return $testCase->execute( |
| 55 | 59 | [ |
@@ -366,7 +366,6 @@ |
||
| 366 | 366 | |
| 367 | 367 | /** |
| 368 | 368 | * @param object $user |
| 369 | - * @param mixed $expected |
|
| 370 | 369 | * |
| 371 | 370 | * @dataProvider getUserProvider |
| 372 | 371 | */ |
@@ -4,9 +4,7 @@ |
||
| 4 | 4 | |
| 5 | 5 | use Doctrine\Common\EventManager; |
| 6 | 6 | use Doctrine\DBAL\Driver\Connection; |
| 7 | - |
|
| 8 | 7 | use Symfony\Component\Filesystem\Filesystem; |
| 9 | - |
|
| 10 | 8 | use Oro\Component\TestUtils\ORM\Mocks\DriverMock; |
| 11 | 9 | use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock; |
| 12 | 10 | |