Whats wrong with this code?

<aside class="col-xs-12 col-sm-12 col-md-3 margin-bottom-20-mobile">
   <div class="service-component">

      <?php
        if($post->post_parent) {
          $titlenamer = get_the_title($post->post_parent);
        }
        else {
          $titlenamer = get_the_title($post->ID);
        }
        ?>
        <h3 class="services-heading"><?php echo $titlenamer; ?> </h3>
        <ul class="list-unstyled vertical-services">
          <?php 
            $list_args = array(
               'post_type'        => 'services',
               'posts_per_page'   => -1
            );
            $list_query = new WP_Query( $list_args );


            // Loop
            if ( $list_query->have_posts() ) : while ( $list_query->have_posts() ) : $list_query->the_post(); ?>
            <li>
              <a href="<?php the_permalink();?>">
                <?php the_title(); ?>
                <span class="glyphicon glyphicon-menu-right service-arrow"></span>
              </a>
            </li>
            <?php else : ?>

          <?php endif; ?>
        </ul>
   </div>
</aside>