mvhub-dev team mailing list archive
-
mvhub-dev team
-
Mailing list archive
-
Message #00137
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