I think both PARAM_FLOAT and PARAM_NUMBER should be deprecated in favour of something else that deals with floating points and commas properly, but I am afraid this will have to wait till new forms library. There should nob be a need for floating point GET parameters...

Thanks Petr, for now I will just make it consistent in code to use PARAM_FLOAT and leave PARAM_NUMBER declaration. Probably, we can get rid of that with something else later.

PARAM_FLOAT declaration is not removed for 3rd party compatibility.

Changes look good thanks Raj, put this up for integration when you are ready and rebased

Thanks Sam and Petr.

Delaying till the start of the 2.4 dev cycle. We don't want any more big changes for now.

Branch re-based,
Pushing it for integration review.

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

Brach re-based.

Just adding Tim here because this touches question component mostly.

https://github.com/rajeshtaneja/moodle/compare/master...wip-mdl-32940#L17R278 is wrong. Should be PARAM_INT.

Otherwise fine.

By the way, http://tracker.moodle.org/browse/MDL-32940?focusedCommentId=156907&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-156907 is really needed for teachers editing numerical questions, but it is a separate issue.

Thanks for the inputs Tim,

Replaced FLOAT with INT at https://github.com/rajeshtaneja/moodle/compare/master...wip-mdl-32940#L17R278

Thanks guys, i've integrated this now

Hi Guys,
Looks good to me.
Passing!
Thanks

yay, it works!

This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

Thank you all for taking the time to get us here.

cheers!