In survey_engine_controllers, there is one huge run() function which I'm
trying to refactor/simplify.
Currently, run() only accept $survey_id and all the data is being stored in
Session, including the page variable. Hence a lot of code is used to jump to
the correct page and serve the correct data.
I propose that we add 1 more parameter = $section_id, so the code could be
more simpler / shorter.
hence run($survey_id) ==> run($survey_id, $section_id)
Is there any disadvantages in doing so ?