Re: ExtendedFileSystemRepository

Your patch has been applied to the CVS, thanks Andy.

-Anthony

Andy Zeneski wrote:
> I got a cvs checkout and tested this change, it does work.
> 
> 
> Index: ExtendedFileSystemRepository.java
> ===================================================================
> RCS file:
> /cvsroot/jpublish/jpublish/src/org/jpublish/repository/filesystem/ExtendedFi
> leSystemRepository.java,v
> retrieving revision 1.23
> diff -u -r1.23 ExtendedFileSystemRepository.java
> --- ExtendedFileSystemRepository.java	12 Feb 2003 21:06:50 -0000	1.23
> +++ ExtendedFileSystemRepository.java	26 Feb 2003 07:26:57 -0000
> @@ -128,6 +128,7 @@
>          try{
>              reader = new BufferedReader(new InputStreamReader(
>                  getInputStream(path), getInputEncoding(path)));
> +            writer = new StringWriter();
> 
>              String name = PathUtilities.makeRepositoryURI(getName(), path);
>              ViewRenderer renderer = siteContext.getViewRenderer();
> @@ -351,9 +352,10 @@
>      Exception{
>          File configFile = pathToConfig(path);
>          if(!configFile.exists()){
> +            log.debug("No config file found; (" +
> configFile.getCanonicalPath() + ") no actions to run");
>              return;
>          }
> -
> +
>          ActionManager actionManager = siteContext.getActionManager();
>          Configuration configuration = new XMLConfiguration(configFile);
> 
> @@ -362,6 +364,7 @@
>          while(contentActionElements.hasNext()){
>              Configuration contentActionElement =
>                  (Configuration)contentActionElements.next();
> +            log.debug("running action - " +
> contentActionElement.getAttribute("name"));
> 
> actionManager.execute(contentActionElement.getAttribute("name"),
>                  context);
>          }
> 
> 
>>-----Original Message-----
>>From: [email protected]
>>[mailto:[email protected]]On Behalf Of Andy
>>Zeneski
>>Sent: Tuesday, February 25, 2003 10:04 PM
>>To: [email protected]
>>Subject: [JPublish-developer] ExtendedFileSystemRepository
>>
>>
>>
>>I believe this class is missing a:
>>
>>writer = new StringWriter();
>>
>>Around line 130 in the get(String, JPublishContext) method.
>>
>>I would have supplied a patch, but I haven't pulled out a CVS copy yet.
>>
>>-Andy
>>
>>
>>
>>-------------------------------------------------------
>>This SF.net email is sponsored by: Scholarships for Techies!
>>Can't afford IT training? All 2003 ictp students receive scholarships.
>>Get hands-on training in Microsoft, Cisco, Sun, Linux/UNIX, and more.
>>www.ictp.com/training/sourceforge.asp
>>_______________________________________________
>>Jpublish-developer mailing list
>>[email protected]
>>https://lists.sourceforge.net/lists/listinfo/jpublish-developer
>>
> 
> 
> 
> 
> 
> -------------------------------------------------------
> This SF.net email is sponsored by: Scholarships for Techies!
> Can't afford IT training? All 2003 ictp students receive scholarships.
> Get hands-on training in Microsoft, Cisco, Sun, Linux/UNIX, and more.
> www.ictp.com/training/sourceforge.asp
> _______________________________________________
> Jpublish-developer mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/jpublish-developer



-------------------------------------------------------
This SF.net email is sponsored by: Scholarships for Techies!
Can't afford IT training? All 2003 ictp students receive scholarships.
Get hands-on training in Microsoft, Cisco, Sun, Linux/UNIX, and more.
www.ictp.com/training/sourceforge.asp