noosfero | removing local acts_as_list dependency (!1550)

[Gabriel Silva]

Gabriel Silva started a new discussion on app/models/article.rb:

>  
>    extend ActsAsHavingImage::ClassMethods
>    acts_as_having_image
> +  acts_as_list scope: :profile

We chose to not implement `acts_as_list` in the article because:

1.  it didn't work with position gaps, and articles may be deleted. To deal with positioning where gaps may appear we are using the `Article.switch_orders`.
2.  it was difficult to set an initial position to all existing articles, and `acts_asl_list` only works if (1) every article has a unique position and (2) there are no gaps.

Maybe those issues only existed on the old version we were using in the `vendor` folder. Would you make sure if it is not a problem with this version?

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/noosfero/noosfero/merge_requests/1550#note_83953674
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listas.softwarelivre.org/pipermail/noosfero-dev/attachments/20180625/d67c6e01/attachment.html>