[Vtigercrm-developers] vtigercrm | PHP 7 compatibility (#197)

[ptdesigner]

Tim, absolutely right!The possible explanation was just a quick fix to alow installation success. I hope that workaround in not definitive because as you said feels dodgy and doesnt solve other areas.

Best regards.


-------- Mensagem original --------
De : Tim Mohrbach <noreply at code.vtiger.com> 
Data: 10/05/2016  09:19  (GMT+00:00) 
Para: ptdesigner at gmail.com 
Assunto: Re: vtigercrm | PHP 7 compatibility (#197) 




@prasad: Good stuff, thanks for trying hard and succeeding with PHP7! 
Regarding your fix in db3ef7f9, yes this will make split work. But I don't like it at all... it's dodgy and just working around the problem instead of tackling it properly. Instead of bloating up vtiger with code that will make deprecated code work, why not change the original cause of the problem and get rid of all split in vtiger code? 

I hope you can comment why you chose this approach to fix things in this particular case... Looking forward to it.







—


View it on GitLab
{"@context":"http://schema.org","@type":"EmailMessage","action":{"@type":"ViewAction","name":"View Issue","url":"http://code.vtiger.com/vtiger/vtigercrm/issues/197#note_1256"}}
You're receiving this notification because you are a member of the vtiger / vtigercrm project team.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20160510/dad5994c/attachment-0001.html>