From 05301e586ea00bec3c6f4b025c3cc00ec169e892 Mon Sep 17 00:00:00 2001 From: Ben Atkins Date: Wed, 23 Jan 2013 15:56:10 -0500 Subject: [PATCH] Refactoring the serializers to be modules instead of classes, since they never get instantiated. --- README.md | 2 +- lib/paper_trail/serializers/json.rb | 2 +- lib/paper_trail/serializers/yaml.rb | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 362e06a3..ed277084 100644 --- a/README.md +++ b/README.md @@ -749,7 +749,7 @@ By default, PaperTrail stores your changes as a YAML dump. You can override this >> PaperTrail.config.serializer = MyCustomSerializer ``` -The serializer needs to be a class that has a `load` and `dump` class method. These serializers are included in the gem for your convenience: +A valid serializer is a `module` (or `class`) that defines a `load` and `dump` method. These serializers are included in the gem for your convenience: * [Yaml](https://github.com/airblade/paper_trail/blob/master/lib/paper_trail/serializers/yaml.rb) - Default * [Json](https://github.com/airblade/paper_trail/blob/master/lib/paper_trail/serializers/json.rb) diff --git a/lib/paper_trail/serializers/json.rb b/lib/paper_trail/serializers/json.rb index 6b5de8e9..5fee0a59 100644 --- a/lib/paper_trail/serializers/json.rb +++ b/lib/paper_trail/serializers/json.rb @@ -2,7 +2,7 @@ require 'active_support/json' module PaperTrail module Serializers - class Json + module Json def self.load(string) ActiveSupport::JSON.decode string end diff --git a/lib/paper_trail/serializers/yaml.rb b/lib/paper_trail/serializers/yaml.rb index 24de8438..139b1a0a 100644 --- a/lib/paper_trail/serializers/yaml.rb +++ b/lib/paper_trail/serializers/yaml.rb @@ -2,7 +2,7 @@ require 'yaml' module PaperTrail module Serializers - class Yaml + module Yaml def self.load(string) YAML.load string end