2010-11-12 5 views
2

Я ожидал, что это будет объединение всех изменений между 2 версиями. (С использованием VESTAL версии 1.0.2)Vestal Versions changes_between не работает для меня

ruby-1.8.7-p174 > contact.version 
=> 12 
ruby-1.8.7-p174 > contact.latest_approved_version 
=> 8 
ruby-1.8.7-p174 > contact.changes 
=> {} 
ruby-1.8.7-p174 > contact.versions.last.changes 
=> {"first_name"=>["I changed this one baby", "AdminF"]} 
ruby-1.8.7-p174 > contact.changes_between(8,12) 
=> {} 
ruby-1.8.7-p174 > contact.changes_between(9,12) 
=> {"deleted"=>[true, false]} 
ruby-1.8.7-p174 > contact.changes_between(10,12) 
=> {} 
ruby-1.8.7-p174 > contact.changes_between(11,12) 
=> {"first_name"=>["I changed this one baby", "AdminF"]} 

Как вы можете видеть, никаких изменений между 8 и 12, но изменения между некоторыми из них.

Странно, что я клянусь, что это работало вчера!

Вот метод девственница использует, я не уверен, что проблема заключается в:

def changes_between(from, to) 
    from_number, to_number = versions.number_at(from), versions.number_at(to) 
    return {} if from_number == to_number 
    chain = versions.between(from_number, to_number).reject(&:initial?) 
    return {} if chain.empty? 

    backward = from_number > to_number 
    backward ? chain.pop : chain.shift unless from_number == 1 || to_number == 1 

    chain.inject({}) do |changes, version| 
     changes.append_changes!(backward ? version.changes.reverse_changes : version.changes) 
    end 
    end 

ответ

0

Вполне возможно, что нет ничего плохого в этом примере вы предоставили. Если, например, нет различий между версиями 8 и 12. Несмотря на то, что были изменения в версиях 9, 10 и 11, метод changes_between не будет показывать никаких изменений, если атрибуты модели были возвращены к тем же значениям.

Я думаю, что самый простой способ проверить, что это проверить:

>> contact.revert_to(8) 
=> "8" 
>> contact.inspect 
=> "#<Contact ... >" 
>> contact.revert_to(12) 
=> "12" 
>> contact.inspect 
=> "#<Contact ... >" 

А потом сравните результат. Я не могу думать о какой-либо другой причине для результата, который вы описали.

Кстати, вызов, который вы сделали для «last_approved_version», это то, что вы вручную добавили в свою модель или это что-то встроенное в vestal_versions? Потому что я также использую 1.0.2 vestal_version, и я не могу найти никакой ссылки на это. Так что если это действительно что-то встроенное, то у нас, вероятно, есть разные вилки vestal_version, и это может быть причиной ...

+0

Да, я добавил последнюю одобренную версию. Если вы видите мой вывод на консоли, вы можете увидеть изменения между каждой версией, поэтому im 99.99 процентов уверены, что изменения существуют. См. Изменения между 11-12 – Joelio

+0

. Все, что я говорю, заключается в том, что, глядя на ваш вывод, возможно, что версии 8 и 12 выглядят точно так же. Если вы можете опубликовать результат из change_between (8,9), changes_between (9,10), changes_between (10,11) и changes_between (11,12), вероятно, было бы легче следить за изменениями, внесенными в каждую версию. И, как я уже упоминал ранее, простой осмотр легко выявит правду. – DanneManne

+0

Человек ваш правый, спасибо, я думаю, очевидным был ответ :) – Joelio