← Back to team overview

mvhub-dev team mailing list archive

Re: [Merge] lp:~priya/mvhub/move_sql_lib_headings into lp:mvhub

 

Review: Needs Fixing
Thanks for fixes and clear explanation of trunk problems. 

This line looks broken though:

+ my $stmt = 'CATEGORY_HEADING_CATEGORY_X_FETCH_HEADING_CATEGORY';

Probably you want to pass the constant into the get_sql_select_statemnet()

Also, it would be swell if $stmt was $sql like it is everywhere else. 
-- 
https://code.launchpad.net/~priya/mvhub/move_sql_lib_headings/+merge/24446
Your team MVHub Developers is subscribed to branch lp:mvhub.



References