Я ожидал, что это будет объединение всех изменений между 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
Да, я добавил последнюю одобренную версию. Если вы видите мой вывод на консоли, вы можете увидеть изменения между каждой версией, поэтому im 99.99 процентов уверены, что изменения существуют. См. Изменения между 11-12 – Joelio
. Все, что я говорю, заключается в том, что, глядя на ваш вывод, возможно, что версии 8 и 12 выглядят точно так же. Если вы можете опубликовать результат из change_between (8,9), changes_between (9,10), changes_between (10,11) и changes_between (11,12), вероятно, было бы легче следить за изменениями, внесенными в каждую версию. И, как я уже упоминал ранее, простой осмотр легко выявит правду. – DanneManne
Человек ваш правый, спасибо, я думаю, очевидным был ответ :) – Joelio