-
Notifications
You must be signed in to change notification settings - Fork 6
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
Review the internal data structures in class Query #158
base: develop
Are you sure you want to change the base?
Commits on Aug 23, 2024
-
Configuration menu - View commit details
-
Copy full SHA for ca98b69 - Browse repository at this point
Copy the full SHA ca98b69View commit details -
Add helper classes OrderItem and ConditionItem to represent the items
in the ORDER BY and the WHERE clause respectively
Configuration menu - View commit details
-
Copy full SHA for c67d44a - Browse repository at this point
Copy the full SHA c67d44aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 811de80 - Browse repository at this point
Copy the full SHA 811de80View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4dec2ed - Browse repository at this point
Copy the full SHA 4dec2edView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9e27157 - Browse repository at this point
Copy the full SHA 9e27157View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4d58e3d - Browse repository at this point
Copy the full SHA 4d58e3dView commit details
Commits on Aug 24, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 397c453 - Browse repository at this point
Copy the full SHA 397c453View commit details -
Configuration menu - View commit details
-
Copy full SHA for c6c0c52 - Browse repository at this point
Copy the full SHA c6c0c52View commit details
Commits on Aug 26, 2024
-
Update the source code creating queries to use the new format of the
conditions argument. Not yet covered: tests, documentation examples.
Configuration menu - View commit details
-
Copy full SHA for f5014cb - Browse repository at this point
Copy the full SHA f5014cbView commit details -
Emit a deprecation warning when passing a mapping in the conditions
argument to Query.addConditions()
Configuration menu - View commit details
-
Copy full SHA for 2fc5f82 - Browse repository at this point
Copy the full SHA 2fc5f82View commit details -
Update the test_06_query.py to use the new format of the conditions
argument when creating queries. Not yet done: verify that evaluating the formal string representation of Query works as expected, add tests for the legacy use of a mapping and verify that the deprecation warning is indeed being raised.
Configuration menu - View commit details
-
Copy full SHA for e47ae77 - Browse repository at this point
Copy the full SHA e47ae77View commit details
Commits on Aug 27, 2024
-
- Update all tests to use the new format of the conditions argument
when creating queries - While we are at it: eliminate a few more occurences of the old legacy ICAT query syntax in the tests
Configuration menu - View commit details
-
Copy full SHA for ada4100 - Browse repository at this point
Copy the full SHA ada4100View commit details -
Add checks to verify that evaluating the formal string representation
operator of Query may indeed be used to create an equivalent query to the tests in test_06_query.py where it makes sense
Configuration menu - View commit details
-
Copy full SHA for 50a1063 - Browse repository at this point
Copy the full SHA 50a1063View commit details
Commits on Aug 28, 2024
-
Add tests for the legacy use of passing a mapping in the conditions
argument in class Query. Also verify that a DeprecationWarning is raised.
Configuration menu - View commit details
-
Copy full SHA for 383c975 - Browse repository at this point
Copy the full SHA 383c975View commit details -
Update example scripts to use the new format of the conditions
argument when creating queries. Eliminate a few more occurrences of the old legacy ICAT query syntax on the way.
Configuration menu - View commit details
-
Copy full SHA for cfc9099 - Browse repository at this point
Copy the full SHA cfc9099View commit details -
Update the tutotial section "Searching for objects in the ICAT server"
to use the new format of the conditions argument when creating queries.
Configuration menu - View commit details
-
Copy full SHA for 60024f9 - Browse repository at this point
Copy the full SHA 60024f9View commit details
Commits on Aug 29, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 38decc0 - Browse repository at this point
Copy the full SHA 38decc0View commit details -
Update the tutotial section "Upload and download files to and from
IDS" to use the new format of the conditions argument when creating queries.
Configuration menu - View commit details
-
Copy full SHA for 93e1a12 - Browse repository at this point
Copy the full SHA 93e1a12View commit details -
Configuration menu - View commit details
-
Copy full SHA for f325880 - Browse repository at this point
Copy the full SHA f325880View commit details -
Add a test using a query where one attributes appears more than once
in the ORDER BY clause
Configuration menu - View commit details
-
Copy full SHA for 587718a - Browse repository at this point
Copy the full SHA 587718aView commit details -
Fixup cfc9099: still missed one occurrence of a mapping in the
conditions argument
Configuration menu - View commit details
-
Copy full SHA for 5ed6692 - Browse repository at this point
Copy the full SHA 5ed6692View commit details -
Configuration menu - View commit details
-
Copy full SHA for e9a545f - Browse repository at this point
Copy the full SHA e9a545fView commit details
Commits on Aug 30, 2024
-
Configuration menu - View commit details
-
Copy full SHA for b358006 - Browse repository at this point
Copy the full SHA b358006View commit details -
Review the order of methods in class Query to be consistent with the
order of the keyword arguments and the order of calling the methods during initialization. This should essentially only have an impact on the presentation of the module reference page in the documentation.
Configuration menu - View commit details
-
Copy full SHA for 8e1e872 - Browse repository at this point
Copy the full SHA 8e1e872View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5c7e114 - Browse repository at this point
Copy the full SHA 5c7e114View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6db1195 - Browse repository at this point
Copy the full SHA 6db1195View commit details -
Add a comment in test_09_deprecations.py that testing the legacy use
of the conditions argument is done in test_06_query.py
Configuration menu - View commit details
-
Copy full SHA for 4025818 - Browse repository at this point
Copy the full SHA 4025818View commit details