I don't know if we should keep them, I don't have a strong opinion but if we decide to keep them they should be maintained and updated (otherwise they should go): my comment was simply to add some background.
Jacopo On Tue, Jun 14, 2016 at 4:22 PM, Taher Alkhateeb <[email protected] > wrote: > Hi Jacopo, > > Does that mean we should keep them? Mind you some of them have incorrect > code (not importing EntityQuery for example). I'm not sure we want to keep > dead code in trunk, unless people are using it? > > Regards, > > Taher Alkhateeb > > On Tue, Jun 14, 2016 at 5:18 PM, Jacopo Cappellato < > [email protected]> wrote: > > > Hi Taher, > > > > some background: these classes depends on jar files that have an > > incompatible license that cannot be distributed with OFBiz; that is why > > they are there but they are not built. > > > > Jacopo > > > > On Tue, Jun 14, 2016 at 4:10 PM, Taher Alkhateeb < > > [email protected] > > > wrote: > > > > > Hi Everyone, > > > > > > I cannot actually believe it but while I was working on a project (I > will > > > announce later) I discovered in the process that the below files cannot > > > compile!!! They existed for years in the code base without even being > > able > > > to compile. They reference non existent libraries or they have faulty > > code > > > (e.g. not importing used code) > > > > > > I propose to delete them immediately from trunk > > > > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/cybersource/IcsPaymentServices.java > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/ideal/IdealEvents.java > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/ideal/IdealPaymentServiceTest.java > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/orbital/OrbitalPaymentServices.java > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/paypal/PayPalServices.java > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/securepay/SecurePayPaymentServices.java > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/securepay/SecurePayServiceTest.java > > > > > > > > > applications/accounting/src/org/ofbiz/accounting/thirdparty/verisign/PayflowPro.java > > > > > > > > > applications/content/src/org/ofbiz/content/openoffice/OpenOfficeByteArrayInputStream.java > > > > > > > > > applications/content/src/org/ofbiz/content/openoffice/OpenOfficeByteArrayOutputStream.java > > > > > > > > > applications/content/src/org/ofbiz/content/openoffice/OpenOfficeServices.java > > > > > > applications/content/src/org/ofbiz/content/openoffice/OpenOfficeWorker.java > > > applications/content/src/org/ofbiz/content/report > > > > > > > > > applications/content/src/org/ofbiz/content/report/JREntityListIteratorDataSource.java > > > > > > > > > applications/content/src/org/ofbiz/content/report/JRMapCollectionDataSource.java > > > applications/order/src/org/ofbiz/order/thirdparty/taxware > > > > > > > > > applications/order/src/org/ofbiz/order/thirdparty/taxware/TaxwareException.java > > > > > > > > > applications/order/src/org/ofbiz/order/thirdparty/taxware/TaxwareServices.java > > > > applications/order/src/org/ofbiz/order/thirdparty/taxware/TaxwareUTL.java > > > applications/product/src/ShipmentScaleApplet.java > > > > > > > > > applications/securityext/src/org/ofbiz/securityext/thirdparty/truition/TruitionCoReg.java > > > > > > > > > framework/webapp/src/org/ofbiz/webapp/view/JasperReportsJXlsViewHandler.java > > > > > > framework/webapp/src/org/ofbiz/webapp/view/JasperReportsPdfViewHandler.java > > > > > > > > > framework/webapp/src/org/ofbiz/webapp/view/JasperReportsPoiXlsViewHandler.java > > > > > > framework/webapp/src/org/ofbiz/webapp/view/JasperReportsXmlViewHandler.java > > > > > > Regards, > > > > > > Taher Alkhateeb > > > > > >
