-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Code cleanup: remove RawData query module #1454
Comments
I think first step 2 should be done, both for core and external plugins, then some time latter step 1. Otherwise, few important tool will cease working ( including civet and DICOM converters) |
Right. I didn't think about the plugins. So yes, first we do 2. We commit and deploy. Once we know for sure the CBRAIN codebase and the plugins don't use the raw_X() methods anymore, we perform 1. |
Just go ahead! |
Important!! You missed a few files in the CBRAIN code base:
For your next pull request, make sure to update these files AND remove the |
Also, make your PR with |
In the early days of CBRAIN we used an ActiveRecord library that wasn't as powerful as the current one.
In order to do some direct column extraction, I created a module that provides these two methods:
These two methods are completely unnecessary anymore, because ActiveRecord has the method
pluck()
which does almost exactly the same thing.This issue is about:
BrainPortal/lib/cbrain_extensions/active_record_extensions/relation_extensions/raw_data.rb
raw_first_column()
andraw_rows()
withpluck()
instead.Some code adjustment might be necessary given the
raw_xxx()
methods can be invoked without any arguments, unlikepluck()
.Checklist over the next few days:
The text was updated successfully, but these errors were encountered: