These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | |||
3 | namespace Kunstmaan\NodeBundle\Helper; |
||
4 | |||
5 | use Doctrine\ORM\EntityManager; |
||
6 | use Kunstmaan\AdminBundle\Helper\DomainConfigurationInterface; |
||
7 | use Kunstmaan\NodeBundle\Validation\URLValidator; |
||
8 | use Psr\Log\LoggerInterface; |
||
9 | use Symfony\Component\Routing\RouterInterface; |
||
10 | |||
11 | /** |
||
12 | * A helper for replacing url's |
||
13 | */ |
||
14 | class URLHelper |
||
15 | { |
||
16 | use URLValidator; |
||
17 | |||
18 | /** |
||
19 | * @var EntityManager |
||
20 | */ |
||
21 | private $em; |
||
22 | |||
23 | /** |
||
24 | * @var RouterInterface |
||
25 | */ |
||
26 | private $router; |
||
27 | |||
28 | /** |
||
29 | * @var LoggerInterface |
||
30 | */ |
||
31 | private $logger; |
||
32 | |||
33 | /** @var array */ |
||
34 | private $nodeTranslationCache = []; |
||
35 | |||
36 | /** @var array */ |
||
37 | private $mediaCache = []; |
||
38 | |||
39 | /** |
||
40 | * @var DomainConfigurationInterface |
||
41 | */ |
||
42 | private $domainConfiguration; |
||
43 | |||
44 | /** |
||
45 | * @param EntityManager $em |
||
46 | * @param RouterInterface $router |
||
47 | * @param LoggerInterface $logger |
||
48 | * @param DomainConfigurationInterface $domainConfiguration |
||
49 | */ |
||
50 | 3 | public function __construct(EntityManager $em, RouterInterface $router, LoggerInterface $logger, DomainConfigurationInterface $domainConfiguration) |
|
0 ignored issues
–
show
|
|||
51 | { |
||
52 | 3 | $this->em = $em; |
|
53 | 3 | $this->router = $router; |
|
54 | 3 | $this->logger = $logger; |
|
55 | 3 | $this->domainConfiguration = $domainConfiguration; |
|
56 | 3 | } |
|
57 | |||
58 | /** |
||
59 | * Replace a given text, according to the node translation id and the multidomain site id. |
||
60 | * |
||
61 | * @param string $text |
||
62 | * |
||
63 | * @return string |
||
64 | */ |
||
65 | 3 | public function replaceUrl($text) |
|
66 | { |
||
67 | 3 | if ($this->isEmailAddress($text)) { |
|
68 | 1 | $text = sprintf('%s:%s', 'mailto', $text); |
|
69 | } |
||
70 | |||
71 | 3 | if ($this->isInternalLink($text)) { |
|
72 | 1 | preg_match_all("/\[(([a-z_A-Z\.]+):)?NT([0-9]+)\]/", $text, $matches, PREG_SET_ORDER); |
|
73 | |||
74 | 1 | if (\count($matches) > 0) { |
|
75 | 1 | foreach ($matches as $match) { |
|
76 | 1 | $fullTag = $match[0]; |
|
77 | 1 | $hostId = $match[2]; |
|
78 | |||
79 | 1 | $hostConfig = !empty($hostId) ? $this->domainConfiguration->getFullHostById($hostId) : null; |
|
80 | 1 | $host = null !== $hostConfig && array_key_exists('host', $hostConfig) ? $hostConfig['host'] : null; |
|
81 | 1 | $hostBaseUrl = $this->domainConfiguration->getHostBaseUrl($host); |
|
82 | |||
83 | 1 | $nodeTranslationId = $match[3]; |
|
84 | 1 | $nodeTranslation = $this->getNodeTranslation($nodeTranslationId); |
|
85 | |||
86 | 1 | if ($nodeTranslation) { |
|
0 ignored issues
–
show
The expression
$nodeTranslation of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.
This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using ![]() |
|||
87 | 1 | $urlParams = ['url' => $nodeTranslation['url']]; |
|
88 | // Only add locale if multilingual site |
||
89 | 1 | if ($this->domainConfiguration->isMultiLanguage($host)) { |
|
90 | $urlParams['_locale'] = $nodeTranslation['lang']; |
||
91 | } |
||
92 | |||
93 | // Only add other site, when having this. |
||
94 | 1 | if ($hostId) { |
|
95 | $urlParams['otherSite'] = $hostId; |
||
96 | } |
||
97 | |||
98 | 1 | $url = $this->router->generate('_slug', $urlParams); |
|
99 | |||
100 | 1 | $text = str_replace($fullTag, $hostId ? $hostBaseUrl . $url : $url, $text); |
|
101 | } else { |
||
102 | $this->logger->error('No NodeTranslation found in the database when replacing url tag ' . $fullTag); |
||
103 | } |
||
104 | } |
||
105 | } |
||
106 | } |
||
107 | |||
108 | 3 | if ($this->isInternalMediaLink($text)) { |
|
109 | 1 | preg_match_all("/\[(([a-z_A-Z]+):)?M([0-9]+)\]/", $text, $matches, PREG_SET_ORDER); |
|
110 | |||
111 | 1 | if (\count($matches) > 0) { |
|
112 | 1 | foreach ($matches as $match) { |
|
113 | 1 | $fullTag = $match[0]; |
|
114 | 1 | $mediaId = $match[3]; |
|
115 | |||
116 | 1 | $mediaItem = $this->getMedia($mediaId); |
|
117 | 1 | if ($mediaItem) { |
|
0 ignored issues
–
show
The expression
$mediaItem of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.
This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using ![]() |
|||
118 | 1 | $text = str_replace($fullTag, $mediaItem['url'], $text); |
|
119 | } else { |
||
120 | $this->logger->error('No Media found in the database when replacing url tag ' . $fullTag); |
||
121 | } |
||
122 | } |
||
123 | } |
||
124 | } |
||
125 | |||
126 | 3 | return $text; |
|
127 | } |
||
128 | |||
129 | 1 | private function getNodeTranslation($nodeTranslationId): array |
|
130 | { |
||
131 | 1 | if (isset($this->nodeTranslationCache[$nodeTranslationId])) { |
|
132 | 1 | return $this->nodeTranslationCache[$nodeTranslationId]; |
|
133 | } |
||
134 | |||
135 | 1 | $stmt = $this->em->getConnection()->executeQuery( |
|
136 | 1 | 'SELECT url, lang FROM kuma_node_translations WHERE id = :nodeTranslationId', |
|
137 | 1 | ['nodeTranslationId' => $nodeTranslationId] |
|
138 | ); |
||
139 | 1 | $nodeTranslation = $stmt->fetch(); |
|
140 | |||
141 | 1 | $this->nodeTranslationCache[$nodeTranslationId] = $nodeTranslation; |
|
142 | |||
143 | 1 | return $nodeTranslation; |
|
144 | } |
||
145 | |||
146 | 1 | private function getMedia($mediaId): array |
|
147 | { |
||
148 | 1 | if (isset($this->mediaCache[$mediaId])) { |
|
149 | 1 | return $this->mediaCache[$mediaId]; |
|
150 | } |
||
151 | |||
152 | 1 | $stmt = $this->em->getConnection()->executeQuery( |
|
153 | 1 | 'SELECT url FROM kuma_media WHERE id = :mediaId', |
|
154 | 1 | ['mediaId' => $mediaId] |
|
155 | ); |
||
156 | 1 | $media = $stmt->fetch(); |
|
157 | |||
158 | 1 | $this->mediaCache[$mediaId] = $media; |
|
159 | |||
160 | 1 | return $media; |
|
161 | } |
||
162 | } |
||
163 |
The
EntityManager
might become unusable for example if a transaction is rolled back and it gets closed. Let’s assume that somewhere in your application, or in a third-party library, there is code such as the following:If that code throws an exception and the
EntityManager
is closed. Any other code which depends on the same instance of theEntityManager
during this request will fail.On the other hand, if you instead inject the
ManagerRegistry
, thegetManager()
method guarantees that you will always get a usable manager instance.