Enhancement Request. Include support for Composer (Discuss)

There are many third-party PHP libraries and modules which now will not install without using Composer. I'm not really that familiar with it but I believe it would be helpful to support it as it seems more and more code is moving to this way of installation and management.

+1 for this as already proposed in !68 (closed) almost a year ago. I don't know any PHP project that is not using composer as a third party dependency management. Even really outdated PHP 6.3 projects have adapted this a long time ago. There is zero reason that composer still has not made it's way into vtiger. I suggest we take one of the many libraries included as an example and prepare another PR... Let me have a look what could be used easily

+1

+1

Added discussion label

mentioned in issue #218 (closed)

If we fix a library but patch does not get pulled then how would composer let us handle override? Is there a way we can lock the version or warn user about bug-fix to apply?

what do you mean by "we fix a library"? I believe we shouldn't change the code of libraries created by third parties (Smarty for example). We should just use composer to establish what version we want.

We have added custom-fix to adodb library to make the installation compatible to XML spec that was supported by adodb (but not any more. Ex: adding foreign constraint or altering table support). In such case, adodb latest from composer would not be helpful and we have to delink off composer (this is my understanding).

removed time estimate

changed milestone to %8.0.0

changed title from vt7 dev release - Enhancement Request. Include support for Composer (Discuss) to Enhancement Request. Include support for Composer (Discuss)

Created branch forcomposer to enable the support.

Pushed changes required for

• Smarty, Rss, htmlpurifier - 060d5325
• TCPDF - 6434fc6b

added enhancement label

removed discussion label

mentioned in merge request !896 (merged)

mentioned in commit cae995d8

closed with merge request !896 (merged)

mentioned in merge request !68 (closed)