Ming,
Your patch solves only the DebugReports problem, while Leonardo's patch solves any difference between what UI expects and backend sets. I was planing to send screenshot patch later after all of us agree with
2014/2/27 21:47, Paulo Ricardo Paz Vital: the method to fix DebugReports, anyway this is another issue not directly linked to my point. My point was that: 1) when one people already worked on something, the later one should communicate with the former one to avoid duplicate effort 2) The patch should have some soaking time to be merged, say 24 hours, and people in other timezone can get an opportunity to review it.
In addition, IMO the point you mentioned about not expose host file system to the front is the root cause of this bug. Many UI paths were broken because the paths set up by backend was not used or followed.
In most cases ,we should use relative path to hide the host file systems from the front users. Only if it must, the absolute path can be used. I don't like the idea to change all the relative static path to absolute ones. IMO, we can only use absolute path for the debugreport and screenshot path. Even better, we can take more effort to re-organize the path to have a reasonable root for all of the path including debugreport and screenshot paths.
Best regards,