Skip to content

Commit 02a058b

Browse files
committed
refactor: remove redundant intermediate variables in Detector classes
Simplify code by directly accessing API from extractor property instead of using intermediate variable across multiple adapter detectors
1 parent 6a43526 commit 02a058b

File tree

19 files changed

+19
-38
lines changed

19 files changed

+19
-38
lines changed

src/Adapters/Archive/Detectors/AuthorName.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ class AuthorName extends Detector
1212
{
1313
public function detect(): ?string
1414
{
15-
$extractor = $this->extractor;
16-
$api = $extractor->getApi();
15+
$api = $this->extractor->getApi();
1716

1817
$result = $api->str('metadata', 'creator');
1918
return (is_string($result) && trim($result) !== '') ? $result : parent::detect();

src/Adapters/Archive/Detectors/Description.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ class Description extends Detector
1212
{
1313
public function detect(): ?string
1414
{
15-
$extractor = $this->extractor;
16-
$api = $extractor->getApi();
15+
$api = $this->extractor->getApi();
1716

1817
$result = $api->str('metadata', 'extract');
1918
return (is_string($result) && trim($result) !== '') ? $result : parent::detect();

src/Adapters/Archive/Detectors/PublishedTime.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,7 @@ class PublishedTime extends Detector
1313
{
1414
public function detect(): ?DateTime
1515
{
16-
$extractor = $this->extractor;
17-
$api = $extractor->getApi();
16+
$api = $this->extractor->getApi();
1817

1918
$fields = ['publicdate', 'addeddate', 'date'];
2019
foreach ($fields as $field) {

src/Adapters/Archive/Detectors/Title.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ class Title extends Detector
1212
{
1313
public function detect(): ?string
1414
{
15-
$extractor = $this->extractor;
16-
$api = $extractor->getApi();
15+
$api = $this->extractor->getApi();
1716

1817
$result = $api->str('metadata', 'title');
1918
return (is_string($result) && trim($result) !== '') ? $result : parent::detect();

src/Adapters/Gist/Detectors/AuthorName.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ class AuthorName extends Detector
1212
{
1313
public function detect(): ?string
1414
{
15-
$extractor = $this->extractor;
16-
$api = $extractor->getApi();
15+
$api = $this->extractor->getApi();
1716

1817
$result = $api->str('owner');
1918
return (is_string($result) && trim($result) !== '') ? $result : parent::detect();

src/Adapters/Gist/Detectors/Code.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,7 @@ public function detect(): ?EmbedCode
2020

2121
private function fallback(): ?EmbedCode
2222
{
23-
$extractor = $this->extractor;
24-
$api = $extractor->getApi();
23+
$api = $this->extractor->getApi();
2524

2625
$code = $api->html('div');
2726
$stylesheet = $api->str('stylesheet');

src/Adapters/Gist/Detectors/PublishedTime.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,7 @@ class PublishedTime extends Detector
1313
{
1414
public function detect(): ?DateTime
1515
{
16-
$extractor = $this->extractor;
17-
$api = $extractor->getApi();
16+
$api = $this->extractor->getApi();
1817

1918
$result = $api->time('created_at');
2019
return $result !== null ? $result : parent::detect();

src/Adapters/ImageShack/Detectors/AuthorName.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,7 @@ class AuthorName extends Detector
1313
{
1414
public function detect(): ?string
1515
{
16-
$extractor = $this->extractor;
17-
$api = $extractor->getApi();
16+
$api = $this->extractor->getApi();
1817

1918
$result = $api->str('owner', 'username');
2019
return (is_string($result) && trim($result) !== '') ? $result : parent::detect();

src/Adapters/ImageShack/Detectors/Description.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ class Description extends Detector
1212
{
1313
public function detect(): ?string
1414
{
15-
$extractor = $this->extractor;
16-
$api = $extractor->getApi();
15+
$api = $this->extractor->getApi();
1716

1817
$result = $api->str('description');
1918
return (is_string($result) && trim($result) !== '') ? $result : parent::detect();

src/Adapters/ImageShack/Detectors/Image.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,7 @@ class Image extends Detector
1313
{
1414
public function detect(): ?UriInterface
1515
{
16-
$extractor = $this->extractor;
17-
$api = $extractor->getApi();
16+
$api = $this->extractor->getApi();
1817

1918
$result = $api->url('direct_link');
2019
return $result !== null ? $result : parent::detect();

0 commit comments

Comments
 (0)