Completed
Push — master ( 97e40f...89ec5c )
by André
40:26 queued 12:35
created

TrashService::loadTrashItem()   A

Complexity

Conditions 3
Paths 3

Size

Total Lines 14
Code Lines 8

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 3
eloc 8
nc 3
nop 1
dl 0
loc 14
rs 9.4285
c 0
b 0
f 0
1
<?php
2
3
/**
4
 * File containing the eZ\Publish\Core\Repository\TrashService class.
5
 *
6
 * @copyright Copyright (C) eZ Systems AS. All rights reserved.
7
 * @license For full copyright and license information view LICENSE file distributed with this source code.
8
 */
9
namespace eZ\Publish\Core\Repository;
10
11
use eZ\Publish\API\Repository\TrashService as TrashServiceInterface;
12
use eZ\Publish\API\Repository\Repository as RepositoryInterface;
13
use eZ\Publish\SPI\Persistence\Handler;
14
use eZ\Publish\API\Repository\Values\Content\Location;
15
use eZ\Publish\Core\Repository\Values\Content\TrashItem;
16
use eZ\Publish\API\Repository\Values\Content\TrashItem as APITrashItem;
17
use eZ\Publish\API\Repository\Values\Content\Query;
18
use eZ\Publish\SPI\Persistence\Content\Location\Trashed;
19
use eZ\Publish\Core\Base\Exceptions\InvalidArgumentValue;
20
use eZ\Publish\Core\Base\Exceptions\UnauthorizedException;
21
use eZ\Publish\API\Repository\Values\Content\Trash\SearchResult;
22
use eZ\Publish\API\Repository\Values\Content\Query\Criterion;
23
use eZ\Publish\API\Repository\Values\Content\Query\SortClause;
24
use DateTime;
25
use Exception;
26
27
/**
28
 * Trash service, used for managing trashed content.
29
 */
30
class TrashService implements TrashServiceInterface
31
{
32
    /**
33
     * @var \eZ\Publish\Core\Repository\Repository
34
     */
35
    protected $repository;
36
37
    /**
38
     * @var \eZ\Publish\SPI\Persistence\Handler
39
     */
40
    protected $persistenceHandler;
41
42
    /**
43
     * @var array
44
     */
45
    protected $settings;
46
47
    /**
48
     * @var \eZ\Publish\Core\Repository\Helper\NameSchemaService
49
     */
50
    protected $nameSchemaService;
51
52
    /**
53
     * Setups service with reference to repository object that created it & corresponding handler.
54
     *
55
     * @param \eZ\Publish\API\Repository\Repository $repository
56
     * @param \eZ\Publish\SPI\Persistence\Handler $handler
57
     * @param \eZ\Publish\Core\Repository\Helper\NameSchemaService $nameSchemaService
58
     * @param array $settings
59
     */
60
    public function __construct(
61
        RepositoryInterface $repository,
62
        Handler $handler,
63
        Helper\NameSchemaService $nameSchemaService,
64
        array $settings = array()
65
    ) {
66
        $this->repository = $repository;
0 ignored issues
show
Documentation Bug introduced by
$repository is of type object<eZ\Publish\API\Repository\Repository>, but the property $repository was declared to be of type object<eZ\Publish\Core\Repository\Repository>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly.

Either this assignment is in error or an instanceof check should be added for that assignment.

class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
Loading history...
67
        $this->persistenceHandler = $handler;
68
        $this->nameSchemaService = $nameSchemaService;
69
        // Union makes sure default settings are ignored if provided in argument
70
        $this->settings = $settings + array(
71
            //'defaultSetting' => array(),
72
        );
73
    }
74
75
    /**
76
     * Loads a trashed location object from its $id.
77
     *
78
     * Note that $id is identical to original location, which has been previously trashed
79
     *
80
     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException if the user is not allowed to read the trashed location
81
     * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException - if the location with the given id does not exist
82
     *
83
     * @param mixed $trashItemId
84
     *
85
     * @return \eZ\Publish\API\Repository\Values\Content\TrashItem
86
     */
87
    public function loadTrashItem($trashItemId)
88
    {
89
        if ($this->repository->hasAccess('content', 'restore') !== true) {
0 ignored issues
show
Deprecated Code introduced by
The method eZ\Publish\Core\Repository\Repository::hasAccess() has been deprecated with message: since 6.6, to be removed. Use PermissionResolver::hasAccess() instead. Check if user has access to a given module / function. Low level function, use canUser instead if you have objects to check against.

This method has been deprecated. The supplier of the class has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead.

Loading history...
90
            throw new UnauthorizedException('content', 'restore');
91
        }
92
93
        $spiTrashItem = $this->persistenceHandler->trashHandler()->loadTrashItem($trashItemId);
94
        $trash = $this->buildDomainTrashItemObject($spiTrashItem);
95
        if (!$this->repository->canUser('content', 'read', $trash->getContentInfo())) {
0 ignored issues
show
Deprecated Code introduced by
The method eZ\Publish\Core\Repository\Repository::canUser() has been deprecated with message: since 6.6, to be removed. Use PermissionResolver::canUser() instead. Check if user has access to a given action on a given value object. Indicates if the current user is allowed to perform an action given by the function on the given
objects.

This method has been deprecated. The supplier of the class has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead.

Loading history...
96
            throw new UnauthorizedException('content', 'read');
97
        }
98
99
        return $trash;
100
    }
101
102
    /**
103
     * Sends $location and all its children to trash and returns the corresponding trash item.
104
     *
105
     * The current user may not have access to the returned trash item, check before using it.
106
     * Content is left untouched.
107
     *
108
     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException if the user is not allowed to trash the given location
109
     *
110
     * @param \eZ\Publish\API\Repository\Values\Content\Location $location
111
     *
112
     * @return null|\eZ\Publish\API\Repository\Values\Content\TrashItem null if location was deleted, otherwise TrashItem
113
     */
114
    public function trash(Location $location)
115
    {
116
        if (!is_numeric($location->id)) {
117
            throw new InvalidArgumentValue('id', $location->id, 'Location');
118
        }
119
120
        if ($this->repository->canUser('content', 'manage_locations', $location->getContentInfo(), $location) !== true) {
0 ignored issues
show
Deprecated Code introduced by
The method eZ\Publish\Core\Repository\Repository::canUser() has been deprecated with message: since 6.6, to be removed. Use PermissionResolver::canUser() instead. Check if user has access to a given action on a given value object. Indicates if the current user is allowed to perform an action given by the function on the given
objects.

This method has been deprecated. The supplier of the class has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead.

Loading history...
121
            throw new UnauthorizedException('content', 'manage_locations');
122
        }
123
124
        $this->repository->beginTransaction();
125
        try {
126
            $spiTrashItem = $this->persistenceHandler->trashHandler()->trashSubtree($location->id);
127
            $this->persistenceHandler->urlAliasHandler()->locationDeleted($location->id);
128
            $this->repository->commit();
129
        } catch (Exception $e) {
130
            $this->repository->rollback();
131
            throw $e;
132
        }
133
134
        // Use sudo as we want a trash item regardless of user access to the trash.
135
        try {
136
            return isset($spiTrashItem)
137
                ? $this->repository->sudo(
138
                    function () use ($spiTrashItem) {
139
                        return $this->buildDomainTrashItemObject($spiTrashItem);
140
                    }
141
                )
142
                : null;
143
        } catch (Exception $e) {
144
            return null;
145
        }
146
    }
147
148
    /**
149
     * Recovers the $trashedLocation at its original place if possible.
150
     *
151
     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException if the user is not allowed to recover the trash item at the parent location location
152
     *
153
     * If $newParentLocation is provided, $trashedLocation will be restored under it.
154
     *
155
     * @param \eZ\Publish\API\Repository\Values\Content\TrashItem $trashItem
156
     * @param \eZ\Publish\API\Repository\Values\Content\Location $newParentLocation
157
     *
158
     * @return \eZ\Publish\API\Repository\Values\Content\Location the newly created or recovered location
159
     */
160
    public function recover(APITrashItem $trashItem, Location $newParentLocation = null)
161
    {
162
        if (!is_numeric($trashItem->id)) {
0 ignored issues
show
Documentation introduced by
The property $id is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
163
            throw new InvalidArgumentValue('id', $trashItem->id, 'TrashItem');
0 ignored issues
show
Documentation introduced by
The property $id is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
164
        }
165
166
        if ($newParentLocation === null && !is_numeric($trashItem->parentLocationId)) {
0 ignored issues
show
Documentation introduced by
The property $parentLocationId is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
167
            throw new InvalidArgumentValue('parentLocationId', $trashItem->parentLocationId, 'TrashItem');
0 ignored issues
show
Documentation introduced by
The property $parentLocationId is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
168
        }
169
170
        if ($newParentLocation !== null && !is_numeric($newParentLocation->id)) {
171
            throw new InvalidArgumentValue('parentLocationId', $newParentLocation->id, 'Location');
172
        }
173
174
        if ($this->repository->hasAccess('content', 'restore') !== true) {
0 ignored issues
show
Deprecated Code introduced by
The method eZ\Publish\Core\Repository\Repository::hasAccess() has been deprecated with message: since 6.6, to be removed. Use PermissionResolver::hasAccess() instead. Check if user has access to a given module / function. Low level function, use canUser instead if you have objects to check against.

This method has been deprecated. The supplier of the class has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead.

Loading history...
175
            throw new UnauthorizedException('content', 'restore');
176
        }
177
178
        $this->repository->beginTransaction();
179
        try {
180
            $newParentLocationId = $newParentLocation ? $newParentLocation->id : $trashItem->parentLocationId;
0 ignored issues
show
Documentation introduced by
The property $parentLocationId is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
181
            $newLocationId = $this->persistenceHandler->trashHandler()->recover(
182
                $trashItem->id,
0 ignored issues
show
Documentation introduced by
The property $id is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
183
                $newParentLocationId
184
            );
185
186
            $content = $this->repository->getContentService()->loadContent($trashItem->contentId);
187
            $urlAliasNames = $this->nameSchemaService->resolveUrlAliasSchema($content);
188
189
            // Publish URL aliases for recovered location
190
            foreach ($urlAliasNames as $languageCode => $name) {
191
                $this->persistenceHandler->urlAliasHandler()->publishUrlAliasForLocation(
192
                    $newLocationId,
193
                    $newParentLocationId,
194
                    $name,
195
                    $languageCode,
196
                    $content->contentInfo->alwaysAvailable
197
                );
198
            }
199
200
            $this->repository->commit();
201
        } catch (Exception $e) {
202
            $this->repository->rollback();
203
            throw $e;
204
        }
205
206
        return $this->repository->getLocationService()->loadLocation($newLocationId);
207
    }
208
209
    /**
210
     * Empties trash.
211
     *
212
     * All locations contained in the trash will be removed. Content objects will be removed
213
     * if all locations of the content are gone.
214
     *
215
     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException if the user is not allowed to empty the trash
216
     */
217 View Code Duplication
    public function emptyTrash()
218
    {
219
        if ($this->repository->hasAccess('content', 'cleantrash') !== true) {
0 ignored issues
show
Deprecated Code introduced by
The method eZ\Publish\Core\Repository\Repository::hasAccess() has been deprecated with message: since 6.6, to be removed. Use PermissionResolver::hasAccess() instead. Check if user has access to a given module / function. Low level function, use canUser instead if you have objects to check against.

This method has been deprecated. The supplier of the class has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead.

Loading history...
220
            throw new UnauthorizedException('content', 'cleantrash');
221
        }
222
223
        $this->repository->beginTransaction();
224
        try {
225
            // Persistence layer takes care of deleting content objects
226
            $this->persistenceHandler->trashHandler()->emptyTrash();
227
            $this->repository->commit();
228
        } catch (Exception $e) {
229
            $this->repository->rollback();
230
            throw $e;
231
        }
232
    }
233
234
    /**
235
     * Deletes a trash item.
236
     *
237
     * The corresponding content object will be removed
238
     *
239
     * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException if the user is not allowed to delete this trash item
240
     *
241
     * @param \eZ\Publish\API\Repository\Values\Content\TrashItem $trashItem
242
     */
243 View Code Duplication
    public function deleteTrashItem(APITrashItem $trashItem)
244
    {
245
        if ($this->repository->hasAccess('content', 'cleantrash') !== true) {
0 ignored issues
show
Deprecated Code introduced by
The method eZ\Publish\Core\Repository\Repository::hasAccess() has been deprecated with message: since 6.6, to be removed. Use PermissionResolver::hasAccess() instead. Check if user has access to a given module / function. Low level function, use canUser instead if you have objects to check against.

This method has been deprecated. The supplier of the class has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead.

Loading history...
246
            throw new UnauthorizedException('content', 'cleantrash');
247
        }
248
249
        if (!is_numeric($trashItem->id)) {
0 ignored issues
show
Documentation introduced by
The property $id is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
250
            throw new InvalidArgumentValue('id', $trashItem->id, 'TrashItem');
0 ignored issues
show
Documentation introduced by
The property $id is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
251
        }
252
253
        $this->repository->beginTransaction();
254
        try {
255
            $this->persistenceHandler->trashHandler()->deleteTrashItem($trashItem->id);
0 ignored issues
show
Documentation introduced by
The property $id is declared protected in eZ\Publish\API\Repository\Values\Content\Location. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

Since your code implements the magic setter _set, this function will be called for any write access on an undefined variable. You can add the @property annotation to your class or interface to document the existence of this variable.

<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

Since the property has write access only, you can use the @property-write annotation instead.

Of course, you may also just have mistyped another name, in which case you should fix the error.

See also the PhpDoc documentation for @property.

Loading history...
256
            $this->repository->commit();
257
        } catch (Exception $e) {
258
            $this->repository->rollback();
259
            throw $e;
260
        }
261
    }
262
263
    /**
264
     * Returns a collection of Trashed locations contained in the trash, which are readable by the current user.
265
     *
266
     * $query allows to filter/sort the elements to be contained in the collection.
267
     *
268
     * @param \eZ\Publish\API\Repository\Values\Content\Query $query
269
     *
270
     * @return \eZ\Publish\API\Repository\Values\Content\Trash\SearchResult
271
     */
272
    public function findTrashItems(Query $query)
273
    {
274
        if ($query->filter !== null && !$query->filter instanceof Criterion) {
275
            throw new InvalidArgumentValue('query->filter', $query->filter, 'Query');
276
        }
277
278
        if ($query->sortClauses !== null) {
279
            if (!is_array($query->sortClauses)) {
280
                throw new InvalidArgumentValue('query->sortClauses', $query->sortClauses, 'Query');
281
            }
282
283
            foreach ($query->sortClauses as $sortClause) {
284
                if (!$sortClause instanceof SortClause) {
285
                    throw new InvalidArgumentValue('query->sortClauses', 'only instances of SortClause class are allowed');
286
                }
287
            }
288
        }
289
290 View Code Duplication
        if ($query->offset !== null && !is_numeric($query->offset)) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
291
            throw new InvalidArgumentValue('query->offset', $query->offset, 'Query');
292
        }
293
294 View Code Duplication
        if ($query->limit !== null && !is_numeric($query->limit)) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
295
            throw new InvalidArgumentValue('query->limit', $query->limit, 'Query');
296
        }
297
298
        $spiTrashItems = $this->persistenceHandler->trashHandler()->findTrashItems(
299
            $query->filter !== null ? $query->filter : null,
300
            $query->offset !== null && $query->offset > 0 ? (int)$query->offset : 0,
301
            $query->limit !== null && $query->limit >= 1 ? (int)$query->limit : null,
302
            $query->sortClauses !== null ? $query->sortClauses : null
0 ignored issues
show
Bug introduced by
It seems like $query->sortClauses !== ...ery->sortClauses : null can also be of type array; however, eZ\Publish\SPI\Persisten...ndler::findTrashItems() does only seem to accept null|array<integer,objec...tent\Query\SortClause>>, maybe add an additional type check?

If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:

/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.

Loading history...
303
        );
304
305
        $trashItems = array();
306
        foreach ($spiTrashItems as $spiTrashItem) {
307
            try {
308
                $trashItems[] = $this->buildDomainTrashItemObject($spiTrashItem);
309
            } catch (UnauthorizedException $e) {
310
                // Do nothing, thus exclude items the current user doesn't have read access to.
311
            }
312
        }
313
314
        $searchResult = new SearchResult();
315
        $searchResult->totalCount = $searchResult->count = count($trashItems);
0 ignored issues
show
Deprecated Code introduced by
The property eZ\Publish\API\Repositor...sh\SearchResult::$count has been deprecated with message: Property is here purely for BC with 5.x/6.x.

This property has been deprecated. The supplier of the class has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the property will be removed from the class and what other property to use instead.

Loading history...
316
        $searchResult->items = $trashItems;
317
318
        return $searchResult;
319
    }
320
321
    /**
322
     * Builds the domain TrashItem object from provided persistence trash item.
323
     *
324
     * @param \eZ\Publish\SPI\Persistence\Content\Location\Trashed $spiTrashItem
325
     *
326
     * @return \eZ\Publish\API\Repository\Values\Content\TrashItem
327
     */
328 View Code Duplication
    protected function buildDomainTrashItemObject(Trashed $spiTrashItem)
329
    {
330
        return new TrashItem(
331
            array(
332
                'contentInfo' => $this->repository->getContentService()->loadContentInfo($spiTrashItem->contentId),
333
                'id' => $spiTrashItem->id,
334
                'priority' => $spiTrashItem->priority,
335
                'hidden' => $spiTrashItem->hidden,
336
                'invisible' => $spiTrashItem->invisible,
337
                'remoteId' => $spiTrashItem->remoteId,
338
                'parentLocationId' => $spiTrashItem->parentId,
339
                'pathString' => $spiTrashItem->pathString,
340
                'depth' => $spiTrashItem->depth,
341
                'sortField' => $spiTrashItem->sortField,
342
                'sortOrder' => $spiTrashItem->sortOrder,
343
            )
344
        );
345
    }
346
347
    /**
348
     * @param int $timestamp
349
     *
350
     * @return \DateTime
351
     */
352
    protected function getDateTime($timestamp)
353
    {
354
        $dateTime = new DateTime();
355
        $dateTime->setTimestamp($timestamp);
356
357
        return $dateTime;
358
    }
359
}
360