Changeset 8411

Timestamp:

03/26/08 23:15:09 (5 years ago)

Author:

damegane

Message:

/lang/ruby/fastladder_mobile .

Location:

lang/ruby/fastladder_mobile/trunk/app

Files:

• controllers/mobile_controller.rb (modified) (4 diffs)
• views/mobile/_item.rhtml (modified) (1 diff)
• views/mobile/_subs.rhtml (added)
• views/mobile/index.rhtml (modified) (1 diff)

lang/ruby/fastladder_mobile/trunk/app/controllers/mobile_controller.rb

@@ -2 +2 @@
   verify :session => :member, :redirect_to => "/mobile/login", :except => :login
   after_filter OutputCompressionFilter
+
+  # 一度に表示するフィード/記事の上限に関する設定
+  MAX_FEED_COUNT = 20 # フィードの件数
+  # 合計の件数とサイズは越えた時点でループを止めるため設定値=上限ではない
+  MAX_TOTAL_ITEM_COUNT = 20 # 記事の件数の合計、0なら無制限
+  MAX_TOTAL_ITEM_SIZE = 30000 # 記事本文のHTMLとしての文字数の合計、0なら無制限
 
   def login
@@ -21 +27 @@
 
   def index
-    conditions = ["has_unread = ?", true]
-    @subs = member.subscriptions.find(:first, :conditions => conditions,
-                                  :order => "subscriptions.rate DESC, subscriptions.id ASC",
-                                  :include => [:folder, { :feed => :crawl_status }])
-    @items = get_items @subs
+    feed_count = 0
+    item_count = 0
+    item_size = 0
+    @subs = get_unread_subs
+    @items = Array.new
+
+    if @subs.size > 0
+      for feed in @subs
+        items = get_items(feed)
+        item_count += items.length
+        item_size += items.inject(0) {|t, i| t + i.body.size }
+        @items << items
+        feed_count += 1
+
+        if item_count >= MAX_TOTAL_ITEM_COUNT ||
+           item_size >= MAX_TOTAL_ITEM_SIZE
+          break
+        end
+      end
+    else
+      render :text => "No unread feeds."
+      return
+    end
+
+    if @subs.size > feed_count
+      @subs = @subs.slice(0, feed_count)
+    end
+    @dbg = {:f => feed_count, :i => item_count, :s => item_size }
   end
 
@@ -31 +60 @@
     # set pins
     if params[:pins]
-      params[:pins].each { | p |
+      params[:pins].each { |p|
         a = p.split("##")
         set_pin(a[0], a[1])
       }
     end
+
     # set readed
-    touch params[:subs_id]
+    if params[:subs_ids]
+      params[:subs_ids].each { |id|
+        touch id
+      }
+    end
+
     # redirect or end
     if params[:next_feed]
@@ -59 +94 @@
 
   private
+  def get_unread_subs
+    conditions = ["has_unread = ?", true]
+    limit = MAX_FEED_COUNT >= 1 ? MAX_FEED_COUNT : 0
+    return member.subscriptions.find(:all, :conditions => conditions,
+                                     :order => "subscriptions.rate DESC, subscriptions.id ASC",
+                                     :include => [:folder, { :feed => :crawl_status }],
+                                     :limit => limit)
+  end
+
   def get_items(subs)
     conditions = subs.viewed_on ? ["stored_on >= ?", subs.viewed_on] : nil
-    items = subs.feed.items.find(:all, :conditions => conditions, :order => "created_on DESC, id DESC", :limit => MAX_UNREAD_COUNT)
-    result = {
-      :subscribe_id => @id,
-      :channel => subs.feed,
-      :items => items,
-    }
-    if items.length > 0
-      result[:last_stored_on] = items.max { |a, b| a.stored_on <=> b.stored_on }.stored_on
-    end
-    result[:ignore_notify] = 1 if subs.ignore_notify
-    return result
+    return subs.feed.items.find(:all, :conditions => conditions, :order => "created_on DESC, id DESC", :limit => MAX_UNREAD_COUNT)
   end
 

lang/ruby/fastladder_mobile/trunk/app/views/mobile/_item.rhtml

@@ -8 +8 @@
       <tr>
         <td>
-          <label for="chk_<%= item_counter %>">
-            <input id="chk_<%= item_counter %>" type="checkbox" name="pins[]" value="<%=h item.link + "##" + item.title%>">
+          <label for="chk_<%= subs_id %>-<%= item_counter %>">
+            <input id="chk_<%= subs_id %>-<%= item_counter %>" type="checkbox" name="pins[]" value="<%=h item.link + "##" + item.title%>">
             Pin
           </label>

lang/ruby/fastladder_mobile/trunk/app/views/mobile/index.rhtml

@@ -1 +1 @@
 <% form_tag('/mobile/next', :method => :post) do %>
-  <div class="title">
-    <h1><%= link_to @subs.feed.title, @subs.feed.link %></h1>
-    <p>
-      <%= @items[:items].size %> new item(s).
-      Rate : <%= @subs.rate %>
-      <%= submit_tag 'Go Next Feed', {:id => "next_feed", :name => "next_feed"}%>
-      <%= submit_tag 'Send Pin Only', {:id => "send_pin", :name => "send_pin"}%>
-    </p>
-  </div>
 
-  <%= render(:partial => "item", :collection => @items[:items]) %>
-  <%= hidden_field_tag 'subs_id', @subs.id %>
+  <% @subs.each_index {|i| %>
+    <%= render(:partial => "subs", :object => @subs[i], :locals => {:item_count => @items[i].length}) %>
+
+    <%= render(:partial => "item", :collection => @items[i], :locals => {:subs_id => @subs[i].id}) %>
+  <% } %>
 
   <div class="navi">
-    <%= submit_tag 'Go Next Feed', {:id => "next_feed", :name => "next_feed"}%>
-    <%= submit_tag 'Send Pin Only', {:id => "send_pin", :name => "send_pin"}%>
+    <%= submit_tag 'Next Feed', {:id => "next_feed", :name => "next_feed"}%>
+    <%= submit_tag 'Pin Only', {:id => "send_pin", :name => "send_pin"}%>
+    <br>
+    Feeds:<%= @dbg[:f] %>
+    Items:<%= @dbg[:i] %>
+    Size:<%= @dbg[:s] %>
   </div>
 <% end %>