[Vtigercrm-developers] VT61 Module installer *still* using 777

[Prasad]

Thanks for the heads up. Will revert back on this next week.

*Connect with us on: *Twitter <http://twitter.com/vtigercrm> *I* Facebook
<http://www.facebook.com/pages/vtiger/226866697333578?sk=wall> *I* Blog
<https://blogs.vtiger.com/>* I* Wiki
<http://wiki.vtiger.com/index.php/Main_Page> *I *Forums
<https://discussions.vtiger.com>*I* Website <https://www.vtiger.com/>


On Fri, Aug 1, 2014 at 1:45 PM, Alan Lord <alanslists at gmail.com> wrote:

> On 31/07/14 12:41, Prasad wrote:
>
>> Alan,
>>
>> I agree - will dive into this any pointer if you have identified would
>> be helpful to fix it.
>>
>
> vtlib/Vtiger/LayoutImport.php line 94
> vtlib/Vtiger/LanguageImport.php line 130
> vtlib/Vtiger/ThemeImport.php line 90
> vtlib/Vtiger/Unzip.php line 45
>
> Firstly I would question why there needs to be 4 separate functions for
> extracting a zip file that are basically copies of each other and do the
> exact same thing.
>
> Secondly, on every Linux distribution I am familiar with there is
> something called the umask[1] which provides the default permissions for
> all new files & directories so I doubt you will need to set chmod anyway...
> Unless it is because the zip archive format does not retain permission
> information. If that's the case then as i said before, all files should be
> 644 (rw-r--r--) and directories 755 (rwxr-x-r-x)
>
> Thirdly - grep the vtiger source tree for "chmod" :-)
>
> Obviously this is all moot for Windows systems but then who uses that old
> OS any more anyway?
>
>
> Cheers
>
> Al
>
> [1] http://en.wikipedia.org/wiki/Umask
>
>
> _______________________________________________
> http://www.vtiger.com/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20140801/54e6ef5c/attachment.html>