@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator

Remove redundant assignment statements

Summary: Both sides are equal, so such assignments have no effect.

Test Plan: Read code; run static code analysis.

Reviewers: O1 Blessed Committers, mainframe98

Reviewed By: O1 Blessed Committers, mainframe98

Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Differential Revision: https://we.phorge.it/D26482

+2 -11
-2
src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php
··· 131 131 132 132 if (is_array($value)) { 133 133 $value = idx($value, 'rule'); 134 - } else { 135 - $value = $value; 136 134 } 137 135 138 136 return phutil_utf8_strtoupper($value);
-2
src/applications/console/plugin/DarkConsoleRequestPlugin.php
··· 99 99 $value = phutil_tag('em', array(), pht('(Masked)')); 100 100 } else if (is_array($value)) { 101 101 $value = @json_encode($value); 102 - } else { 103 - $value = $value; 104 102 } 105 103 106 104 $rows[] = array(
-2
src/applications/differential/storage/DifferentialHunk.php
··· 361 361 switch ($type) { 362 362 case self::DATATYPE_TEXT: 363 363 // In this storage type, the changes are stored on the object. 364 - $data = $data; 365 364 break; 366 365 case self::DATATYPE_FILE: 367 366 $data = $this->loadFileData(); ··· 375 374 switch ($format) { 376 375 case self::DATAFORMAT_RAW: 377 376 // In this format, the changes are stored as-is. 378 - $data = $data; 379 377 break; 380 378 case self::DATAFORMAT_DEFLATED: 381 379 $data = PhabricatorCaches::inflateData($data);
+2 -4
src/applications/files/document/PhabricatorJupyterDocumentEngine.php
··· 728 728 private function readStringList(array $src, $key) { 729 729 $list = idx($src, $key); 730 730 731 - if (is_array($list)) { 732 - $list = $list; 733 - } else if (is_string($list)) { 731 + if (is_string($list)) { 734 732 $list = array($list); 735 - } else { 733 + } else if (!is_array($list)) { 736 734 $list = array(); 737 735 } 738 736
-1
src/applications/meta/query/PhabricatorApplicationQuery.php
··· 157 157 $apps = msort($apps, 'getName'); 158 158 break; 159 159 case self::ORDER_APPLICATION: 160 - $apps = $apps; 161 160 break; 162 161 default: 163 162 throw new Exception(