No "self" needed here

app/controllers/asciicasts_controller.rb

@@ -15,7 +15,6 @@ class AsciicastsController < ApplicationController
   end
 
   def show
-    # TODO: filter out private or not (????)
     respond_to do |format|
       format.html do
         view_counter.increment(asciicast, cookies)

app/models/asciicast.rb

@@ -48,7 +48,7 @@ class Asciicast < ActiveRecord::Base
   end
 
   def self.for_category_ordered(category, order, page = nil, per_page = nil)
-    collection = self.non_private
+    collection = non_private
 
     if category == :featured
       collection = collection.featured