On 2 March 2015 at 15:18, Simioni Tanguy <[email protected]> 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 [email protected].
To post to this group, send email to [email protected].
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.

Reply via email to