Which is a better" coding practice, in this scenario? DRY or Single Level Indentation? ((Or is there a better way I'm just not seeing?))

codeStyleCombat.php

<?php

// DRY 
if ($condition) {
    $output->alterOnce();
    if ($secondCondition) {
        $output->alterMore();
    }
}
return $output

/* ******************************* */

// One level of indentation...
if (!$condition) {
    return $output;
}
$output->alterOnce();
if ($secondCondition) {
    $output->alterAgain();
}
return $output;

While single level indention is cleaner to read, the DRY approach would be what I would do.

if ($condition && !$condition2) {
    $output->alterOnce();
}

if ($condition && $condition) {
    $output->alterOnce();
    $output->alterMore();
}

but i think I would go with second, i don't consider return $output as duplication...

Ideally I would try to forge this into something where ->alter(Collection $changes) so it works with 1 or more, but that's hard to judge without concrete examples.

Thank you both!

There was a bit more code between the innermost cheeky braces, just oversimplified here.

I am using the second format. I like single indentation and no else as mental exercises, as well as easier to read. This situation didn't have an else, but every time i find one in existing code, I try to refactor it.