Code Duplication - sovereignbot/citadel - Measure and Improve Code Quality continuously with Scrutinizer

Code Duplication Length = 13-13 lines in 3 locations

src/Plugins/onTimer/kills.php 3 locations


                foreach ($killData as $kill) {
                    if (isset($kill->killID) && ($kill->killID > $latest)) {
                        switch ($type) {
                            case "character":
                                if ($kill->victim->characterID == $id) {
                                    $msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk | https://beta.eve-kill.net/kill/{$kill->killID}/";
                                    $this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
                                }

                                foreach ($kill->attackers as $attacker) {
                                    if ($attacker->characterID == $id && $attacker->finalBlow == 1) {
                                        $msg = "{$attacker->characterName} participated in killing {$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName} / {$kill->victim->shipTypeName}) in a {$attacker->shipTypeName} doing a total of {$attacker->damageDone} damage, and helped destroy {$kill->totalValue}isk | https://beta.eve-kill.net/kill/{$kill->killID}/";
                                        $this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
                                    }
                                }
                                break;
                            case "corporation":
                                if ($kill->victim->corporationID == $id) {
                                    $msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";

                                    }
                                }
                                break;
                            case "corporation":
                                if ($kill->victim->corporationID == $id) {
                                    $msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";
                                    $this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
                                }

                                foreach ($kill->attackers as $attacker) {
                                    if ($attacker->corporationID == $id && $attacker->finalBlow == 1) {
                                        $msg = "{$attacker->characterName} participated in killing {$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName} / {$kill->victim->shipTypeName}) in a {$attacker->shipTypeName} doing a total of {$attacker->damageDone} damage, and helped destroy {$kill->totalValue}isk | https://beta.eve-kill.net/kill/{$kill->killID}/";
                                        $this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
                                    }
                                }
                                break;
                            case "alliance":
                                if ($kill->victim->allianceID == $id) {
                                    $msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";

                                    }
                                }
                                break;
                            case "alliance":
                                if ($kill->victim->allianceID == $id) {
                                    $msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";
                                    $this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
                                }

                                foreach ($kill->attackers as $attacker) {
                                    if ($attacker->allianceID == $id && $attacker->finalBlow == 1) {
                                        $msg = "{$attacker->characterName} participated in killing {$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName} / {$kill->victim->shipTypeName}) in a {$attacker->shipTypeName} doing a total of {$attacker->damageDone} damage, and helped destroy {$kill->totalValue}isk | https://beta.eve-kill.net/kill/{$kill->killID}/";
                                        $this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
                                    }
                                }
                                break;
                        }
                    }
                }

		@@ 91-103 (lines=13) @@
88		foreach ($killData as $kill) {
89		if (isset($kill->killID) && ($kill->killID > $latest)) {
90		switch ($type) {
91		case "character":
92		if ($kill->victim->characterID == $id) {
93		$msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk \| https://beta.eve-kill.net/kill/{$kill->killID}/";
94		$this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
95		}
96
97		foreach ($kill->attackers as $attacker) {
98		if ($attacker->characterID == $id && $attacker->finalBlow == 1) {
99		$msg = "{$attacker->characterName} participated in killing {$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName} / {$kill->victim->shipTypeName}) in a {$attacker->shipTypeName} doing a total of {$attacker->damageDone} damage, and helped destroy {$kill->totalValue}isk \| https://beta.eve-kill.net/kill/{$kill->killID}/";
100		$this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
101		}
102		}
103		break;
104		case "corporation":
105		if ($kill->victim->corporationID == $id) {
106		$msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";
		@@ 104-116 (lines=13) @@
101		}
102		}
103		break;
104		case "corporation":
105		if ($kill->victim->corporationID == $id) {
106		$msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";
107		$this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
108		}
109
110		foreach ($kill->attackers as $attacker) {
111		if ($attacker->corporationID == $id && $attacker->finalBlow == 1) {
112		$msg = "{$attacker->characterName} participated in killing {$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName} / {$kill->victim->shipTypeName}) in a {$attacker->shipTypeName} doing a total of {$attacker->damageDone} damage, and helped destroy {$kill->totalValue}isk \| https://beta.eve-kill.net/kill/{$kill->killID}/";
113		$this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
114		}
115		}
116		break;
117		case "alliance":
118		if ($kill->victim->allianceID == $id) {
119		$msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";
		@@ 117-129 (lines=13) @@
114		}
115		}
116		break;
117		case "alliance":
118		if ($kill->victim->allianceID == $id) {
119		$msg = "{$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName}) lost {$kill->victim->shipTypeName} in {$kill->solarSystemName} ({$kill->regionName}) with a total value of {$kill->totalValue}isk";
120		$this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
121		}
122
123		foreach ($kill->attackers as $attacker) {
124		if ($attacker->allianceID == $id && $attacker->finalBlow == 1) {
125		$msg = "{$attacker->characterName} participated in killing {$kill->victim->characterName} ({$kill->victim->corporationName} / {$kill->victim->allianceName} / {$kill->victim->shipTypeName}) in a {$attacker->shipTypeName} doing a total of {$attacker->damageDone} damage, and helped destroy {$kill->totalValue}isk \| https://beta.eve-kill.net/kill/{$kill->killID}/";
126		$this->db->execute("UPDATE killmailPosting SET latestKillID = :killID WHERE id = :rowID", array(":killID" => $kill->killID, ":rowID" => $rowID));
127		}
128		}
129		break;
130		}
131		}
132		}

sovereignbot / citadel

Code Duplication Length = 13-13 lines in 3 locations

src/Plugins/onTimer/kills.php 3 locations