fix: check null pointer from master on no-pvp combat

[kaleohanopahala]

Now the code checks for a null target right away to prevent crashes.
The summon condition is moved up, eliminating redundant checks.
Unnecessary null verifications (like if (target && ...)) have been removed.
Validations for tile properties and PvP conditions are reorganized for clearer and more efficient logic.

Note:
Reopened because i accidentally deleted all branches haha.

[myallith]

Github Actions 5 x 0 Leilani :D

Line 355:
const auto targetMonster = target->getMonster();

Change to:
const auto &targetMonster = target->getMonster();

Line 359:
const auto attackerMonster = attacker->getMonster();

Change to:
const auto &attackerMonster = attacker->getMonster();

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud