Struts 2 - File upload - Update of file does not work

[Marcos Maia]

Hi,

I'm using struts 2 and the UploadInterceptor to upload an image for an user account, it's working. The problem is whenever I try to update the uploaded photo it does not work. Struts 2 upload interceptor just ignores my new file and leave the old picture there. The new file is uploaded to same dir and with same name as the old one. Follows a code fragment from interceptor which is called while uploading:

   

public String saveFotografia() throws Exception {
        log.debug("entrou SignupAction3.saveFotografia...");
        //pega ServletContext para definir um diretório para a foto.
        ServletContext sContext = ServletActionContext.getServletContext();
        //checa se o arquivo não é nulo.
        if(foto != null)
        {
        	//direciona as fotos para este diretório.
        	String diretorioFotos = sContext.getRealPath("/images/fotosusuarios/");
        	
        	//Cria o diretório se o mesmo não existir.
        	File dirPath = new File(diretorioFotos);
            
            if (!dirPath.exists())
            {
            	log.debug("diretorio nao existe criando==>> " + diretorioFotos);
                dirPath.mkdirs();
            }
        	
            //salva a foto com o nome do e-mail do usário para evitar duplicidade.
            String nomeFoto = getRequest().getSession().getAttribute("email").toString() + "." + getExtensaoArquivo();
        	File arquivoSalvo = new File(diretorioFotos, nomeFoto);
        	
        	foto.renameTo(arquivoSalvo);
        	log.debug("Content type: " + fotoContentType);
        	//atualizando nome da foto
            user.setFoto(nomeFoto);
            userManager.saveUser(user);
        }
        return SUCCESS;
    }
 

Any clues on this?

[]s

[David Newton]

The upload interceptor deletes the uploaded file; it must be copied to a new location--I don't know if simply renaming it is enough.

[Marcos Maia]

Humm..

Just find a solution for my problem, follows the new fragment of code and considerations:

  
public String saveFotografia() throws Exception {
        log.debug("entrou SignupAction3.saveFotografia...");
        //pega ServletContext para definir um diretório para a foto.
        ServletContext sContext = ServletActionContext.getServletContext();
        //checa se o arquivo não é nulo.
        if(foto != null)
        {
        	//direciona as fotos para este diretório.
        	String diretorioFotos = sContext.getRealPath("/images/fotosusuarios/");
        	
        	//Cria o diretório se o mesmo não existir.
        	File dirPath = new File(diretorioFotos);
            
            if (!dirPath.exists())
            {
            	log.debug("diretorio nao existe criando==>> " + diretorioFotos);
                dirPath.mkdirs();
            }
        	
            //salva a foto com o nome do e-mail do usário para evitar duplicidade.
            String nomeFoto = getRequest().getSession().getAttribute("email").toString() + "." + getExtensaoArquivo();
        	File arquivoSalvo = new File(diretorioFotos, nomeFoto);
        	if(arquivoSalvo.exists())
        	{	log.debug("Arquivo ja existe renomeando...");
        		File arquivoBkp = new File(diretorioFotos, nomeFoto + ".bkp." + getExtensaoArquivo());
        		if(arquivoBkp.exists())
        		{
        			log.debug("removendo bkp de foto de usuario para manter somente 1");
        			boolean deletou = arquivoBkp.delete();
        			log.debug("Conseguiu deletar o arquivo " + arquivoBkp.getName() + "? " + deletou);
        		}
        		arquivoSalvo.renameTo(arquivoBkp);
        		arquivoSalvo = new File(diretorioFotos, nomeFoto);
        	}
        	foto.renameTo(arquivoSalvo);
        	log.debug("Content type: " + fotoContentType);
        	//atualizando nome da foto
            user.setFoto(nomeFoto);
            userManager.saveUser(user);
        }
        return SUCCESS;
    }

  

What I did was adding some code to check if the user had already an uploaded photo and if it has I rename it. Because I only want 1 back up I check if I already have this back up and if I have I delete it before adding the new Photo.

[]s

[Marcos Maia]

Just notice that sometimes the file gets locked by the server and the code I write is not able to delete or rename the files... I'll try something with timestamp but this will cause holding all uploaded photos from user..

Still accepting suggestions on this.

[]s