[Vtigercrm-developers] That "test" directory...

stprasad at stprasad.com stprasad at stprasad.com
Tue Oct 22 12:14:05 UTC 2013


Vikas,
Simple search for the test folder should do it.

grep 101 for the brave.

Let's bite the bullet once and for all.

Regards, ST Prasad 

Sent from BlackBerry® on Airtel

-----Original Message-----
From: Alan Lord <alanslists at gmail.com>
Sender: vtigercrm-developers-bounces at lists.vtigercrm.comDate: Tue, 22 Oct 2013 13:07:08 
To: <vtigercrm-developers at lists.vtigercrm.com>
Reply-To: vtigercrm-developers at lists.vtigercrm.com
Subject: Re: [Vtigercrm-developers] That "test" directory...

On 22/10/13 13:00, Vikas Jain wrote:
> Alan,
>
> include folder was retained as there were some dependency on that
> folder, we are reviewing this and will merge into includes folder.
>
> changing *test *folder name to more meaningful name is planned, but
> there are many references in the code which might take longer time
> to change and evaluate.

I understand - it's just a *really* bad name... The not-so-well-informed 
sysadmin might well just delete it on sight!

Also there are currently three possible locations advertising themselves 
for handling uploads:

/test/upload
/cache/upload
/storage

Which one is correct? I'd get rid of the others or name them more sensibly.

Cheers

Al

>
> On 21 October 2013 18:57, Adam Heinz <amh at metricwise.net
> <mailto:amh at metricwise.net>> wrote:
>
>     My guess is that storage/vtiger.txt exists to make sure that the
>     folder is created when the zip file is extracted.  I deploy vtiger
>     via puppet, so I deleted that stuff ages ago and have puppet create
>     the folder.  It would be similarly trivial to add some sort of
>     "ensure file exists" guard to the PHP logic.
>
>
>     On Mon, Oct 21, 2013 at 5:25 AM, Alan Lord
>     <alanslists at gmail.com
>     <mailto:alanslists at gmail.com>> wrote:
>
>         C'mon guys...
>
>         Before you ship VT6 could you please give that /test directory a
>         proper name?
>
>         Also we have /include & /includes (which is a bit annoying).
>
>         There are two "upload" directories:
>
>             ./test/upload
>             ./cache/upload
>
>
>         And in the /storage directory is a file vtiger.txt that simply
>         says "uploads"... ;-)
>
>
>
>         Al
>
>         _________________________________________________
>         http://www.vtiger.com/
>
>
>
>     _______________________________________________
>     http://www.vtiger.com/
>
>
>
>
> --
> Regards
> Vikas
> Vtiger Team
>
>
> _______________________________________________
> http://www.vtiger.com/
>


_______________________________________________
http://www.vtiger.com/



More information about the vtigercrm-developers mailing list