search

juizmill / ListService

Sistema simples de help desk.
9 stars 4 forks source link

Usar a pasta 'data/uploads' para guardar os uploads #83

Closed danizord closed 11 years ago

danizord commented 11 years ago

Convencionalmente a pasta data é utilizada para guardar esses arquivos "de dados".

Eu não usei a constante DIRECTORY_SEPARATOR porque ela é completamente desnecessária, o / funciona tanto no Windows como nos sistemas *nix. Inclusive seria interessante remover ela do repositório inteiro, oq vcs acham?

juizmill commented 11 years ago

Seria bom se gerar ela automaticamente caso não existir.

danizord commented 11 years ago

@juizmill O LSBase\Utils\UploadFile faz isso.

dexfs commented 11 years ago

não precisa remover. configura no gitignore coloca um .gitkeep ou .gitignore dentro dela. no .gitignore tu coloca no final !.gitignore

eu normalmente deixo a pasta dessa forma.

juizmill commented 11 years ago

O .gitgnore tem que escrever algo no arquivo? Estou perdido agora...

dexfs commented 11 years ago

existe um .gitignore que é o que você coloca o que quer e o que não quer para projeto, esse fica na raiz do projeto. o que fica dentro de pastas que você quer apenas apenas a pasta, esse vai vazio mesmo. por isso eu uso .gitkeep para diferenciar.

juizmill commented 11 years ago

Acho que entendi, se eu não quero que os arquivos de uma determinada pasta não seja enviada para o repositório git eu coloco uma arquivo .gitkeep ou .gitignore dentro desta pasta,

danizord commented 11 years ago

@dexdev Já está sendo feito dessa forma. E eu estava falando sobre remover as constantes, não a pasta.

juizmill commented 11 years ago

Se tirar as constantes não vai dar problema com servidores Windows? Ou tem outra forma que não tenha problemas?

danizord commented 11 years ago

@juizmill Como eu disse na descrição da PR, a constante é desnecessária porque a / funciona no Windows, Mac, Linux, [...], mas isso é assunto para outra PR, essa PR aqui é só para usar a pasta data/uploads para guardar os uploads.

juizmill commented 11 years ago

@danizord O estranho é que eu tinha instalado o sistema em um computador windows e deu problema, quando eu coloquei a constante funcionou normal, mas se vocês acham que é melhor remover, então vamos remover, o código ficará até mais limpo :+1:

danizord commented 11 years ago

@juizmill Blz, nós podemos fazer isso mais pra frente, axo q antes de mecher em qualquer código precisamos fazer os testes pra garantir que tudo continuará funcionando.

Pode fazer o merge.