"All" link browses back to all types even when browsing by type

If you browse by type, such that the url is:

/archive/blog

...and then click on "All" under the date in the Years row, you will now be searching all content types in addition to all years.

The problem comes from the fact that the "All" url is being given the url of 'archive' in the following code, however, all other links are sent through the _archive_url function which appends the /$type to the url.

 $output .= '<li'. ($date->year?'':' class="selected"') .'>'. l(t('All'), 'archive', array('title' => format_plural($all_count, '1 post', '@count posts'))) ."</li>\n";

To fix, simply change the line to:

 $output .= '<li'. ($date->year?'':' class="selected"') .'>'. l(t('All'),        _archive_url($type,NULL,NULL)             , array('title' => format_plural($all_count, '1 post', '@count posts'))) ."</li>\n";