Skip to content
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.

Sign up for GitHub

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

Jump to bottom

replace RawSQL with ORM queries #36

Closed
atodorov opened this issue Jun 7, 2017 · 0 comments
Closed

replace RawSQL with ORM queries #36

atodorov opened this issue Jun 7, 2017 · 0 comments

Comments

@atodorov
Copy link
Member

atodorov commented Jun 7, 2017

No description provided.

atodorov added a commit that referenced this issue Jun 25, 2018
@atodorov
Add linter for raw SQL, refs #36
d823d08
atodorov added a commit that referenced this issue Jun 26, 2018
@atodorov
Add linter for raw SQL, refs #36
a22a558
atodorov added a commit that referenced this issue Jul 1, 2018
@atodorov
Remove some RawSQL. Refs #36
3ee56eb 
- remove RawSQL.num_case_bugs - not in use at all
- replace RawSQL.num_cases with an annotated Count
- replace RawSQL.num_runs with an annotated Count
- replace RawSQL.num_plans with an annotated Count
  NOTE: num_children (aka RawSQL.num_plans) is removed from the
        query used in advanced search b/c I think the value is
        not used at all
atodorov added a commit that referenced this issue Jul 1, 2018
@atodorov
Refactor RawSQL.num_case_run_bugs. Refs #36
7f9ca56 
refactor into annotated Count
atodorov added a commit that referenced this issue Jul 1, 2018
@atodorov
Remove RawSQL queries. Fixes #36
1beedb3 
case_runs_count_by_status_under_run & custom_details_case_run_count
were never used in the code.

the rest of the queries are used in Overall Report and are rewriten
to use a sub-query and annotated counts.
atodorov added a commit that referenced this issue Jul 2, 2018
@atodorov
Remove some RawSQL. Refs #36
6918293 
- remove RawSQL.num_case_bugs - not in use at all
- replace RawSQL.num_cases with an annotated Count
- replace RawSQL.num_runs with an annotated Count
- replace RawSQL.num_plans with an annotated Count
  NOTE: num_children (aka RawSQL.num_plans) is removed from the
        query used in advanced search b/c I think the value is
        not used at all
atodorov added a commit that referenced this issue Jul 2, 2018
@atodorov
Refactor RawSQL.num_case_run_bugs. Refs #36
51b7820 
refactor into annotated Count
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@atodorov