Pierre Rioux

Results 158 comments of Pierre Rioux

Oh, is it a named parameter? Don't use the name, since we're not all using the same versions of Ruby everywhere.

Fixed all the DPs with an explicit ```ruby def read_only? true end ```

https://github.com/aces/cbrain/commit/fab9c9193e176e42f4d8f7f7d41088d3f2cf7a00 and https://github.com/aces/cbrain-plugins-neuro/commit/f8f656d27f486ffe67a031fc040b0998a6908663

The branch has been merged but the issue might still exist.

Yes, these overlay panels are very old and annoying to no end. I created them. They were supposed to be used only for short quick tiny bits of informations (e.g....

This fix should go around: https://github.com/aces/cbrain/blob/00076c82fa94aae43706d48e9cb1673a5eff5f1f/BrainPortal/app/controllers/userfiles_controller.rb#L744 The return value shoudl be checked.