search

damoncourtney / installjammer

InstallJammer Multiplatform Installer
http://www.installjammer.com/
Other
31 stars 20 forks source link

Uninstall errors - File list missing new lines #65

Open ghost opened 13 years ago

ghost commented 13 years ago

If you use the uninstall select files option when uninstalling an application, and if some files are left over, the installer correctly lists these files as an error, but the <%Errors%> file list doesn't have any line breaks in them: So you end up with an unfriendly list of errors as shown below:

C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\web-client\WEB-INF\lib is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\web-client\WEB-INF is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\web-client is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\page-studio\WEB-INF\lib is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\page-studio\WEB-INF is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\page-studio is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\default-webapp\WEB-INF\lib is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\default-webapp\WEB-INF is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps\default-webapp is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\webapps is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\openim\conf is not empty.C:\Program Files\Alterian\Content Manager 7.0.0.306\Server\openim is not

etc.

The easy fix is to add a newline to the Errors list in UninstallSelectedFiles.action:204

      append info(Errors) [sub "<%DirectoryNotEmptyText%>\n"]