Need to combine/eliminate duplicates in nested array

Question

I need to create an accordion list of news article titles grouped by year. So far the contents of the accordion panels are correct with each panel showing all of the news posts for that year.

The problem is that I'm getting multiple panels for years based on the number of articles for the year. So for example year 2003 has three news articles, so I'm getting three accordion panels for 2003, each with the three corresponding news articles for that year. For 2004 I get one accordion panel because there's only one article for 2004. 2006 has six articles so I get six panels each with the six articles inside.

What do I need to do to get only one panel with the correct number of titles inside?

Here's my SQL:

   $newsposts = $wpdb->get_results("
SELECT YEAR(wp_posts.post_date) AS post_year, wp_posts.post_title, wp_posts.ID AS post_id 
FROM wp_posts LEFT JOIN wp_term_relationships ON 
(wp_posts.ID = wp_term_relationships.object_id) 
WHERE wp_term_relationships.term_taxonomy_id = 3 
AND wp_posts.post_status = 'publish' 
ORDER BY post_date
              ", OBJECT );

Here's my markup:

                  <?php foreach($newsposts as $year) : ?> <?php //print_r($year); ?>
                  <li class="accordion-item" data-accordion-item>
                    <a href="#" class="accordion-title"><?php echo $year->post_year; ?></a>

                     <div class="accordion-content" data-tab-content>
                       <?php foreach($newsposts as $post) : ?> <?php //print_r($post); ?>
                       <?php // Continue unless years match
                         if ( $post->post_year != $year->post_year ) continue; ?>
                           <a href="<?php the_permalink($post->post_id); ?>"><?php echo $post->post_title; ?></a><br/>
                       <?php endforeach; //$posts ?>
                     </div>

                   </li>
                 <?php endforeach; //$year ?>
                 </ul>

Somehow I need to limit the first foreach to only one for each year represented. I would appreciate any help.

banacan · Accepted Answer · 2020-04-07 20:56:49Z

I worked through this and found an answer thanks to this post:remove duplicates in foreach

In my case here is my revised markup:

            <?php $yearposts = array(); ?>
            <?php foreach($newsposts as $year) : ?> <?php $postyear = $year->post_year; ?>
            <?php if(!in_array($postyear, $yearposts))  { ?>
               <li class="accordion-item" data-accordion-item>
                <a href="#" class="accordion-title"><?php echo $postyear; array_push($yearposts,$postyear); ?></a>
                   <div class="accordion-content" data-tab-content>
                     <ul class="accordlist">
                       <?php foreach($newsposts as $post) : ?> 
                       <?php // Continue unless years match
                         if ( $post->post_year != $year->post_year ) continue; ?>
                       <li><a class="newstitle" href="<?php the_permalink($post->post_id); ?>"><?php echo $post->post_title; ?></a></li>
                       <?php endforeach; //$posts ?>
                     </ul>
                   </div>
                 </li> 
              <?php } // end in_array conditional ?>
            <?php endforeach; //$year ?>
           </ul>

As the indicated post says,

Basically we are just using an additional array to store the printed values. Each time we see a new value, we print it and add it to the array.

My version is broken into pieces to surround the elements that are required for the accordion formatting.

Anyway, I hope this helps someone else. I needed answers from different questions to find my solution. Now maybe someone will find everything in this one place.

Cheers

Collectives™ on Stack Overflow

Need to combine/eliminate duplicates in nested array

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related