I'm taking over a codeigniter project and notice the original dev uses a mixture of short tag and php tags in the views. For example:

<div id="content">

        <?=show_header()?>

        <ul id="products">
            <?php if (count($products) > 0) : ?>
            <?php foreach($products as $product) : ?>
             ...
</div>

Is this bad practice to inherit? I think it is already causing me problems in my dev environment.