sorted filelist properly, minor performance tweak, less code in the view

main.rb

@@ -61,6 +61,8 @@ def insert_into_pstore(path_to_gpx)
 	@file_list = Dir.glob("gpx/#{params[:foldername]}/*.gpx").sort.reverse
 
 	files_to_insert = Array.new	
+
+	#detect new files
 	@pstore.transaction(true) do
 		@file_list.each do |item|
 			if @pstore[params[:foldername]].nil?
@@ -71,24 +73,27 @@ def insert_into_pstore(path_to_gpx)
 			end #if
 		end #do
 	end #do
-
-
+	#insert new files
 	files_to_insert.each do |gpxfile|
 		insert_into_pstore(gpxfile)
 	end
 
 
-	#update total stats
+	#update total stats and get file_list for view
+	@view_list = Array.new
 	@stats = Hash.new
 	@stats["total_distance"] = 0.0
 	@stats["total_duration"] = 0.0
 	@pstore.transaction do
 		@pstore[params[:foldername]].each do |gpxfile|
 			@stats["total_distance"] += @pstore[gpxfile].distance
 			@stats["total_duration"] += @pstore[gpxfile].duration
+			@view_list << @pstore[gpxfile]
 		end unless @pstore[params[:foldername]].nil?
 	end
 
+	@view_list.sort!{|item1, item2| item2.time_start <=> item1.time_start}
+
 	erb(:category)
 end
 

views/category.erb

@@ -26,18 +26,15 @@ Total distance: <%= (@stats["total_distance"] / 1000).round %> km, Total duratio
 
 <table id="tracktable" border="0">
 <th>filename</th><th>starting time</th><th>duration</th><th>distance</th><th>avg speed</th>
-	<% @pstore.transaction(true) do %>
-		<% @file_list.each_with_index do |f,index| %>
+		<% @view_list.each_with_index do |f,index| %>
 			<tr class="<%= if index.even? then "even";else "odd";end%>">
-				<td><a href="/details/<%= f %>"><%= f.split("/").last[0..-5] %></a></td>
-				<td><%= @pstore[f].time_start.strftime("%d.%h.%y%t-%t%H:%M") %></td>
-				<td><%= (@pstore[f].duration / 60).round %> min</td>
-				<td><%= @pstore[f].distance %> m</td>
-				<td><%= @pstore[f].speed_avg.to_s[/\d*\.\d\d/] %> km/h</td>
+				<td><a href="/details/<%= f.filename %>"><%= f.filename.split("/").last[0..-5] %></a></td>
+				<td><%= f.time_start.strftime("%d.%h.%y%t-%t%H:%M") %></td>
+				<td><%= (f.duration / 60).round %> min</td>
+				<td><%= f.distance %> m</td>
+				<td><%= f.speed_avg.to_s[/\d*\.\d\d/] %> km/h</td>
 			</tr>
 		<% end %>
-	<% end %>
-
 </table>
 	</div>
 </html>