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

[Vikas Jain]

Alan/Prasad,

Already did a grep before replying to this thread(43 mathces) its very easy
to replace, the complications comes for migration.
There are places/features that were referencing test folder which had saved
lots of files, moving them during migration might
break it, also with this all those feature will need re validation.



On 22 October 2013 17:44, <stprasad at stprasad.com> wrote:

> 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/
>
> _______________________________________________
> http://www.vtiger.com/
>



-- 
Regards
Vikas
Vtiger Team
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20131022/4157dcd6/attachment.html>