Ruby on Rails Monday, March 2, 2015

On 2 March 2015 at 15:18, Simioni Tanguy <lists@ruby-forum.com> wrote:
> Colin Law wrote in post #1169465:
> Did you move it
>> from after a private declaration? Actions need to be in public
>> section.
>
> after check , it was in private declaration , i'm learning ruby and
> didn't know private declaration was for all function declare after
> "private" key word.

OK, that explains it. I should have thought of that earlier. I still
suggest not using action names with capital letters though.

>
> Again think for you help , and sorry for my bad english ...

It is not bad at all.

Another issue you might like to think about is that you are using a
very old version of rails. You should at least update to 3.2.21 as
you are missing a number of security fixes. Also version 3.2 will go
out of support within a few months I think and will not get even
security fixes any more. No publicly accessible site should be using
3.2 after that.

Also it is, I think, a bad idea to keep the default route
match ':controller(/:action(/:id(.:format)))'
as that allows get and post access to any action in any controller.
You should explicitly define the routes you actually want and whether
they are GET, POST etc.

Colin

--
You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-talk+unsubscribe@googlegroups.com.
To post to this group, send email to rubyonrails-talk@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/CAL%3D0gLvyBviu0dumaDiR6QZNcwp4X31AbT2rGz9K5T4pejPztg%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

No comments:

Post a Comment