Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-32069

Make it possible to configure help to save to a different directory than temp

    Details

    • 12 status:
      Resolved
    • Test Plan:
      Hide

      Locally add the property
      help.indexpath=${sakai.home}/helpindex

      (You can have this to be any path you want, setting this will put the help in ${CATALINA_BASE}/sakai/helpindex

      • Start up Sakai then click on the Help tool, it might take a few seconds to load
      • Then check the file system to make sure help files are in this new path (in sakai.help) rather than in the default tomcat temp, which is usually ${CATALINA_BASE}/temp/sakai.help
      • If you remove the ${CATALINA_BASE}/temp it shouldn't recreate the help there.

      I'd also check to make sure that if the help is updated that these files are updated. I believe that it is but this might be a little more work to test.

      Show
      Locally add the property help.indexpath=${sakai.home}/helpindex (You can have this to be any path you want, setting this will put the help in ${CATALINA_BASE}/sakai/helpindex Start up Sakai then click on the Help tool, it might take a few seconds to load Then check the file system to make sure help files are in this new path (in sakai.help) rather than in the default tomcat temp, which is usually ${CATALINA_BASE}/temp/sakai.help If you remove the ${CATALINA_BASE}/temp it shouldn't recreate the help there. I'd also check to make sure that if the help is updated that these files are updated. I believe that it is but this might be a little more work to test.

      Description

      The help tool has been hardcoded since the beginning to save it's indexes to java.io.tmpdir. This causes problems if the temp directory is cleaned and help indexes haven't been accessed (if you're cleaning based on access time) and they are not regenerated by the help tool.

      They should be put somewhere else and properly cleaned up/regenerated on restart.

      	private static final String LUCENE_INDEX_PATH = System
      	.getProperty("java.io.tmpdir")
      	+ File.separator + "sakai.help";
      

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                jonespm Matthew Jones
                Reporter:
                jonespm Matthew Jones
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Git Source Code