RetroMocha / obvious

Obvious is a clean architecture.
MIT License
264 stars 16 forks source link

fs_plug.rb is incorrect #23

Closed pencilcheck closed 11 years ago

pencilcheck commented 11 years ago

https://github.com/RetroMocha/obvious_status/blob/master/external/fs_plug.rb

# add data to the list if it's a new element 
input[:id] = max_id + 1
data << input if new_element

This should be

# add data to the list if it's a new element 
if new_element
    input[:id] = max_id + 1
    data << input 
end
brianknapp commented 11 years ago

Yep, good find. I actually have a fix for it already, but I haven't had time to commit it. Hopefully, I'll get to it later today.

brianknapp commented 11 years ago

This is fixed and will be going out in the 0.0.8 release