Hello, community. I'm a beginner in Osclass and PHP. Please I need your help.

I'm trying to create a footer menu based on the selection of cities or regions in descending order of the number of items.

One part of the code displays the selection based on cities, and the other part of the code displays the selection based on regions.

Each of the two parts of the code works if I add them to the page one at a time.

If I add them at the same time, the second code displays white (blank) entries.

Where did I make the mistake? Thank you very much.




<?php $cities = CityStats :: newInstance()->listCities('%%%%', '>', 'i_num_items DESC'); ?>
      <?php $i = 1; ?>
<select name="sCity" id="sCity" onchange="this.options[this.selectedIndex].value && (window.location = this.options[this.selectedIndex].value);">   
<option value="">Select a city</option> 
      <?php foreach($cities as $c) { ?>
        <?php if($i <= 30) { ?>
      <option value="<?php echo osc_search_url(array('page' => 'search', 'sCity' => $c['city_id']));?>"><?php echo $c['city_name'];?> (<?php echo $c['items']; ?>) </option>
          <?php $i++; ?>
        <?php } ?>
      <?php } ?>       
</select>
   
<?php $regions = RegionStats :: newInstance()->listRegions('%%%%', '>', 'i_num_items DESC'); ?>
      <?php $i = 1; ?>
<select name="sRegion" id="sRegion" onchange="this.options[this.selectedIndex].value && (window.location = this.options[this.selectedIndex].value);">   
<option value="">Select a region</option> 
      <?php foreach($regions as $r) { ?>
        <?php if($i <= 30) { ?>
      <option value="<?php echo osc_search_url(array('page' => 'search', 'sRegion' => $r['pk_i_id']));?>"><?php echo $r['s_name']; ?> (<?php echo $r['i_num_items']; ?>)</option>
          <?php $i++; ?>
        <?php } ?>
      <?php } ?>       
</select>

Try to use same naming convention in regions as you did in cities.
s_name ==> region_name
pk_i_id ==> city_id
i_num_items ==> items

Thank you very much, MB Themes!  Now it works properly.

Below is the fully functional code, maybe other forum members need it too:
<?php $cities = CityStats :: newInstance()->listCities('%%%%', '>', 'i_num_items DESC'); ?>
      <?php $i = 1; ?>
<select name="sCity" id="sCity" onchange="this.options[this.selectedIndex].value && (window.location = this.options[this.selectedIndex].value);">
<option value="">Select a city</option> 
      <?php foreach($cities as $c) { ?>
        <?php if($i <= 30) { ?>
<option value="<?php echo osc_search_url(array('page' => 'search', 'sCity' => $c['city_id']));?>"><?php echo $c['city_name'];?> (<?php echo $c['items']; ?>) </option>
          <?php $i++; ?>
        <?php } ?>
      <?php } ?>  
</select>   
<?php $regions = RegionStats :: newInstance()->listRegions('%%%%', '>', 'items DESC'); ?>
      <?php $i = 1; ?>
<select name="sRegion" id="sRegion" onchange="this.options[this.selectedIndex].value && (window.location = this.options[this.selectedIndex].value);">
<option value="">Select a region</option> 
      <?php foreach($regions as $r) { ?>
        <?php if($i <= 30) { ?>
<option value="<?php echo osc_search_url(array('page' => 'search', 'sRegion' => $r['region_id']));?>"><?php echo $r['region_name']; ?> (<?php echo $r['items']; ?>)</option>
          <?php $i++; ?>
        <?php } ?>
      <?php } ?>  
</select>

@Fantasy
Thanks

Thanks for the solution

Can this be done on categories?

<?php
          osc_goto_first_category();
          $i = 1;
        ?>
<select name="sCategory" id="sCategory" onchange="this.options[this.selectedIndex].value && (window.location = this.options[this.selectedIndex].value);">
<option value="">Category</option>
          <?php while(osc_has_categories()) { ?>
            <?php if($i <= 10) { ?>
         <option value="<?php echo osc_search_url(array('page' => 'search', 'sCategory' => osc_category_id())); ?>"><?php echo osc_category_name();?> <?php if(osc_category_total_items()>0){ echo '('.osc_category_total_items().')';} ?></option>          
            <?php } ?>
            <?php $i++; ?>
          <?php } ?>
</select>

<?php
          osc_goto_first_category();
          $i = 1;
        ?>
<select name="sCategory" id="sCategory" onchange="this.options[this.selectedIndex].value && (window.location = this.options[this.selectedIndex].value);">
<option value="">Category</option>
          <?php while(osc_has_categories()) { ?>
            <?php if($i <= 10) { ?>
         <option value="<?php echo osc_search_url(array('page' => 'search', 'sCategory' => osc_category_id())); ?>"><?php echo osc_category_name();?> <?php if(osc_category_total_items()>0){ echo '('.osc_category_total_items().')';} ?></option>          
            <?php } ?>
            <?php $i++; ?>
          <?php } ?>
</select>

Thanks for your quick response and the amazing result.

Excellent it works 

<?php $regions = RegionStats :: newInstance()->listRegions('%%%%', '>', 'items DESC'); ?>
      <?php $i = 1; ?>
<select name="sRegion" id="sRegion" onchange="this.options[this.selectedIndex].value && (window.location = this.options[this.selectedIndex].value);">
<option value="">Select a region</option>
      <?php foreach($regions as $r) { ?>
        <?php if($i <= 30) { ?>
<option value="<?php echo osc_search_url(array('page' => 'search', 'sRegion' => $r['region_id']));?>"><?php echo $r['region_name']; ?> (<?php echo $r['items']; ?>)</option>
          <?php $i++; ?>
        <?php } ?>
      <?php } ?>
</select>
The code for cities works as well, but the code for the regions does not display the regions!
What is the reason in your opinion?

Review region stats table

Review region stats table

Thank you, the site stats have been recalculated and the code is now working fine

thank you again

Cool