From 381e280228365ba7e297fb2c7ca7583ff3fa4e15 Mon Sep 17 00:00:00 2001 From: Sam Date: Tue, 19 Mar 2013 22:13:37 -0700 Subject: [PATCH] we should not be mixing this in to our serializers 1. it gives the class broken methods (like cache in production) 2. it conflicts with am serializers --- app/serializers/basic_topic_serializer.rb | 3 --- app/serializers/listable_topic_serializer.rb | 1 - 2 files changed, 4 deletions(-) diff --git a/app/serializers/basic_topic_serializer.rb b/app/serializers/basic_topic_serializer.rb index 38d956510eb1a2..ec3456eea38f71 100644 --- a/app/serializers/basic_topic_serializer.rb +++ b/app/serializers/basic_topic_serializer.rb @@ -2,8 +2,5 @@ # The most basic attributes of a topic that we need to create a link for it. class BasicTopicSerializer < ApplicationSerializer - include ActionView::Helpers - attributes :id, :fancy_title, :slug - end diff --git a/app/serializers/listable_topic_serializer.rb b/app/serializers/listable_topic_serializer.rb index fd8939798475c4..5ddf500a9462a6 100644 --- a/app/serializers/listable_topic_serializer.rb +++ b/app/serializers/listable_topic_serializer.rb @@ -1,7 +1,6 @@ require_dependency 'age_words' class ListableTopicSerializer < BasicTopicSerializer - include ActionView::Helpers attributes :reply_count, :posts_count,