Skip to content

Commit 400e7fc

Browse files
committed
Fixes for linting
1 parent f48551b commit 400e7fc

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

plugins/faustwp/includes/replacement/callbacks.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,9 +127,9 @@ function image_source_srcset_replacement( $sources ) {
127127
);
128128

129129
foreach ( $sources as $width => $source ) {
130-
131130
if ( ! $replace_media_urls ) {
132-
$sources[ $width ]['url'] = faustwp_replace_urls( $patterns, $wp_site_urls, $source['url'] );;
131+
$sources[ $width ]['url'] = faustwp_replace_urls( $patterns, $wp_site_urls, $source['url'] );
132+
133133
continue;
134134
}
135135

plugins/faustwp/includes/replacement/functions.php

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ function faustwp_get_wp_site_urls() {
141141
$host_url = wp_parse_url( $site_url, PHP_URL_HOST );
142142

143143
if ( ! is_string( $host_url ) ) {
144-
return apply_filters( 'faustwp_get_wp_site_urls', [ $site_url ] );
144+
return apply_filters( 'faustwp_get_wp_site_urls', array( $site_url ) );
145145
}
146146

147147
$is_https = substr( $site_url, 0, 6 ) === 'https:';
@@ -151,7 +151,7 @@ function faustwp_get_wp_site_urls() {
151151
array(
152152
$is_https ? "https://$host_url" : "http://$host_url",
153153
$is_https ? "http://$host_url" : "https://$host_url",
154-
"//$host_url"
154+
"//$host_url",
155155
)
156156
);
157157
}
@@ -188,7 +188,6 @@ function faustwp_get_wp_media_urls() {
188188
function faustwp_get_relative_upload_url( $site_urls ) {
189189
$upload_dir = wp_upload_dir()['baseurl'];
190190

191-
192191
foreach ( $site_urls as $site_url ) {
193192
if ( false !== strpos( $upload_dir, $site_url ) ) {
194193
return (string) str_replace( $site_url, '', $upload_dir );
@@ -202,34 +201,36 @@ function faustwp_get_relative_upload_url( $site_urls ) {
202201
* Replaces the media URL for various media urls
203202
*
204203
* @param string $content The content to be updated with the new media URL.
205-
* @param array $wp_media_urls An array of media URLS.
204+
* @param array $wp_media_urls An array of media URLS.
206205
* @param string $replace_url The media URL to be updated to.
207206
*
208-
* @return string
207+
* @return string The replaced string
209208
*/
210209
function faustwp_replace_media_url( string $content, array $wp_media_urls, string $replace_url ) {
211210
return str_replace( $wp_media_urls, $replace_url, $content );
212211
}
213212

214213

215214
/**
216-
* @param array $patterns
217-
* @param mixed $wp_site_urls
218-
* @param mixed $url
215+
* Replaces urls for multiple patterns
216+
*
217+
* @param array $patterns The array of patterns.
218+
* @param mixed $wp_site_urls The array of site URLs.
219+
* @param mixed $content The content to be updated.
219220
*
220-
* @return array|string|string[]|null
221+
* @return mixed The replaced content
221222
*/
222-
function faustwp_replace_urls( array $patterns, mixed $wp_site_urls, mixed $url ) {
223+
function faustwp_replace_urls( array $patterns, mixed $wp_site_urls, mixed $content ) {
223224
$i = 0;
224225

225226
return preg_replace_callback(
226227
$patterns,
227228
function () use ( &$wp_site_urls, &$i ) {
228229
$replacement = $wp_site_urls[ $i ] . '/';
229-
$i ++;
230+
$i++;
230231

231232
return $replacement;
232233
},
233-
$url
234+
$content
234235
);
235236
}

0 commit comments

Comments
 (0)