← Back to team overview

mvhub-dev team mailing list archive

Re: [Merge] lp:~saroun-sek/mvhub/sql_change into lp:mvhub

 

> get_sql_num_and_percent_up_to_date()
> 
> is used in both reports.pl and admin_reports.pl
> 
> please restore it.

I'm pretty sure I checked before I deleted it.  I don't think it was used in admin_reports.pl

It was used in reports.pl, tho.
But as part of an SQL.

453	-        sql => 'SELECT * FROM ('
454	-            . MVHub::Reports::get_sql_num_and_percent_up_to_date( $dbh,
455	-            'agency', 'agencies', 6 )
456	-            . ') as agency_query, ('
457	-            . MVHub::Reports::get_sql_num_and_percent_up_to_date( $dbh,
458	-            'program', 'programs', 6 )
459	-            . ') as program_query '
460	-
461	+        sql => MVHub::Utils::DB::get_sql_select_statement(
462	+            'AGENCY_PROGRAM_X_SEARCHERS_PER_DAY')

Once I moved that code over, I don't think it was called again.  I'll need to double check.


-- 
https://code.launchpad.net/~saroun-sek/mvhub/sql_change/+merge/28658
Your team MVHub Developers is subscribed to branch lp:mvhub.



References