Re: svn commit: r1841675 - /ofbiz/tools/verify-ofbiz-release.sh

[Michael Brohl]

Hi Mathieu,

thanks for the hint!

There is no portability requirement, I once wrote this script to ease 
the release checks and shared it with the community.


I changed the code so that there is no need for exporting the setting 
anymore.


Regards,

Michael


Am 22.09.18 um 14:41 schrieb Mathieu Lirzin:

Hello Michael,

mbr...@apache.org writes:


  # use english gpg output
-LC_MESSAGES=en_EN.UTF-8
+export LC_MESSAGES=en_EN.UTF-8

This syntax is a bashism.  I don't know the portability requirements for
this script, but I would recommend using this syntax instead:

   LC_MESSAGES=en_EN.UTF-8
   export LC_MESSAGES

Thanks.






smime.p7s
Description: S/MIME Cryptographic Signature

Re: svn commit: r1841675 - /ofbiz/tools/verify-ofbiz-release.sh

[Mathieu Lirzin]

Hello Michael,

mbr...@apache.org writes:

>  # use english gpg output
> -LC_MESSAGES=en_EN.UTF-8
> +export LC_MESSAGES=en_EN.UTF-8

This syntax is a bashism.  I don't know the portability requirements for
this script, but I would recommend using this syntax instead:

  LC_MESSAGES=en_EN.UTF-8
  export LC_MESSAGES

Thanks.

-- 
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761  070D 0ADE E100 9460 4D37

Re: [VOTE] [RELEASE] Apache OFBiz 16.11.05

[Michael Brohl]

+1

Everything is ok:

$ ../ofbiz-tools/verify-ofbiz-release.sh apache-ofbiz-16.11.05.zip
sha check of file: apache-ofbiz-16.11.05.zip
Using sha file: apache-ofbiz-16.11.05.zip.sha512
apache-ofbiz-16.11.05.zip: 9D0C613C C3D88C37 A5A41F53 55F0D6A0 A465AB29 
00987DF6 5070BBA6 4DCA43B4 1C8C44DB E7A1BBA5 4528AE17 04B45DE8 722F702E 
9C13508A 0CD900A1 3F1D3B1B
apache-ofbiz-16.11.05.zip: 9D0C613C C3D88C37 A5A41F53 55F0D6A0 A465AB29 
00987DF6 5070BBA6 4DCA43B4 1C8C44DB E7A1BBA5 4528AE17 04B45DE8 722F702E 
9C13508A 0CD900A1 3F1D3B1B

sha checksum OK

GPG verification output
gpg: Signature made Mon Sep 17 11:02:30 2018 CEST using RSA key ID 847AF9E0
gpg: Good signature from "Jacopo Cappellato (CODE SIGNING KEY) 
" [ultimate]



$ ./gradlew loadDefault testIntegration

...

BUILD SUCCESSFUL


Thanks Jacopo,

Michael

Am 21.09.18 um 16:26 schrieb Jacopo Cappellato:

  This is the vote thread to release a new bug fix release for the
release16.11 branch. This new release, "Apache OFBiz 16.11.05" will
supersede all the previous releases from the same branch.

The release files can be downloaded from here:

https://dist.apache.org/repos/dist/dev/ofbiz/

and are:

* apache-ofbiz-16.11.05.zip
* KEYS: text file with keys
* apache-ofbiz-16.11.05.zip.asc: the detached signature file
* apache-ofbiz-16.11.05.zip.sha512: checksum file

Please download and test the zip file and its signatures (for instructions
on testing the signatures see http://www.apache.org/info/verification.html).

Vote:

[ +1] release as Apache OFBiz 16.11.05
[ -1] do not release

This vote will be open for at least 5 days.

For more details about this process please read
http://www.apache.org/foundation/voting.html

Kind Regards,

Jacopo






smime.p7s
Description: S/MIME Cryptographic Signature

Relation type in SecurityGroupPermission entity

[Deepak Nigam]

Hi Folks,

In the definition of 'SecurityGroupPermission' entity, the relation type
with 'SecurityPermission' entity is 'one-nofk'. Hence the system is
allowing to use the permissions not defined anywhere. (ex.
ORDERMGR_CRQ_ADMIN/ORDERMGR_CRQ_UPDATE).
Do we have any use case for this type of relation?

IMO, here the relation type should be 'one' here to maintain
the referential integrity. WDYT?


Thanks & Regards
--
Deepak Nigam
HotWax Systems Pvt. Ltd.

Re: Upgrading the Authorize.net AIM API

[Pierre Smits]

I believe first the discussion on how we’re going to deal with these kind
of 3rd party solution integrations, whether they be fintech, logistic or
other,  should be completed first.

Many times aspects regarding this has been brought forward in other threads
and in tickets.



Best regards,

Pierre Smits

*Apache Trafodion , Vice President*
*Apache Directory , PMC Member*
Apache Incubator , committer
*Apache OFBiz , contributor (without privileges)
since 2008*
Apache Steve , committer

On Sat, Sep 22, 2018 at 1:39 PM, Nameet Jain  wrote:

> Hello Team,
>
> While researching around Advanced Integration Method (AIM) Authorize.net
> API, I found that the AIM API has now been deprecated. Authorize.net
> suggest upgrading the AIM support to XML or JSON based support. Please let
> me know your thoughts.
>
> Here is the link for your reference
> https://developer.authorize.net/api/upgrade_guide/
> https://developer.authorize.net/api/reference/index.html
>
> Thanks
> --
> Regards,
> Nameet Jain
>

Upgrading the Authorize.net AIM API

[Nameet Jain]

Hello Team,

While researching around Advanced Integration Method (AIM) Authorize.net
API, I found that the AIM API has now been deprecated. Authorize.net
suggest upgrading the AIM support to XML or JSON based support. Please let
me know your thoughts.

Here is the link for your reference
https://developer.authorize.net/api/upgrade_guide/
https://developer.authorize.net/api/reference/index.html

Thanks
--
Regards,
Nameet Jain

Re: [VOTE] [RELEASE] Apache OFBiz 16.11.05

[Aditya Sharma]

+1

Thanks and Regards,

*Aditya Sharma* | Enterprise Software Engineer
HotWax Commerce  by HotWax Systems

[image: https://www.linkedin.com/in/aditya-p-sharma/]



On Sat, Sep 22, 2018 at 4:06 PM Rishi Solanki 
wrote:

> +1
>
> Thanks!
> --
> Rishi Solanki
> Sr Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: +91-9893287847
> http://www.hotwaxsystems.com
> www.hotwax.co
>
>
> On Sat, Sep 22, 2018 at 4:01 PM Swapnil Mane <
> swapnil.m...@hotwaxsystems.com>
> wrote:
>
> > +1
> >
> >
> > - Best Regards,
> > Swapnil M Mane
> >
> > On Fri, Sep 21, 2018 at 7:56 PM Jacopo Cappellato <
> > jacopo.cappell...@hotwaxsystems.com> wrote:
> >
> > >  This is the vote thread to release a new bug fix release for the
> > > release16.11 branch. This new release, "Apache OFBiz 16.11.05" will
> > > supersede all the previous releases from the same branch.
> > >
> > > The release files can be downloaded from here:
> > >
> > > https://dist.apache.org/repos/dist/dev/ofbiz/
> > >
> > > and are:
> > >
> > > * apache-ofbiz-16.11.05.zip
> > > * KEYS: text file with keys
> > > * apache-ofbiz-16.11.05.zip.asc: the detached signature file
> > > * apache-ofbiz-16.11.05.zip.sha512: checksum file
> > >
> > > Please download and test the zip file and its signatures (for
> > instructions
> > > on testing the signatures see
> > http://www.apache.org/info/verification.html
> > > ).
> > >
> > > Vote:
> > >
> > > [ +1] release as Apache OFBiz 16.11.05
> > > [ -1] do not release
> > >
> > > This vote will be open for at least 5 days.
> > >
> > > For more details about this process please read
> > > http://www.apache.org/foundation/voting.html
> > >
> > > Kind Regards,
> > >
> > > Jacopo
> > >
> >
>

Re: [VOTE] [RELEASE] Apache OFBiz 16.11.05

[Rishi Solanki]

+1

Thanks!
--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847
http://www.hotwaxsystems.com
www.hotwax.co


On Sat, Sep 22, 2018 at 4:01 PM Swapnil Mane 
wrote:

> +1
>
>
> - Best Regards,
> Swapnil M Mane
>
> On Fri, Sep 21, 2018 at 7:56 PM Jacopo Cappellato <
> jacopo.cappell...@hotwaxsystems.com> wrote:
>
> >  This is the vote thread to release a new bug fix release for the
> > release16.11 branch. This new release, "Apache OFBiz 16.11.05" will
> > supersede all the previous releases from the same branch.
> >
> > The release files can be downloaded from here:
> >
> > https://dist.apache.org/repos/dist/dev/ofbiz/
> >
> > and are:
> >
> > * apache-ofbiz-16.11.05.zip
> > * KEYS: text file with keys
> > * apache-ofbiz-16.11.05.zip.asc: the detached signature file
> > * apache-ofbiz-16.11.05.zip.sha512: checksum file
> >
> > Please download and test the zip file and its signatures (for
> instructions
> > on testing the signatures see
> http://www.apache.org/info/verification.html
> > ).
> >
> > Vote:
> >
> > [ +1] release as Apache OFBiz 16.11.05
> > [ -1] do not release
> >
> > This vote will be open for at least 5 days.
> >
> > For more details about this process please read
> > http://www.apache.org/foundation/voting.html
> >
> > Kind Regards,
> >
> > Jacopo
> >
>

Re: Demo Trunk NPE

[Swapnil Mane]

Hi team,

We have already Jira for this [1].

[1] https://issues.apache.org/jira/browse/OFBIZ-10342


- Best Regards,
Swapnil M Mane

On Sat, Sep 22, 2018 at 3:48 PM Girish Vasmatkar <
girish.vasmat...@hotwaxsystems.com> wrote:

> Hi Rishi/Mohammad
>
> This looks an intermittent issue as I could not reproduce it. However, in
> webtools, under Fetch Logs, I tried searching for "NullPointer" in the
> error.log file and it did return some instances of NullPointerException.
>
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
> java.lang.IllegalArgumentException: Error running script at location
> [component://webtools/groovyScripts/service/Services.groovy]:
> java.lang.NullPointerException
> Caused by: java.lang.NullPointerException
>
> Best,
> Girish Vasmatkar
> HotWax Systems
>
> On Sat, Sep 22, 2018 at 3:02 PM Mohammad Kathawala <
> mohammad.kathaw...@hotwaxsystems.com> wrote:
>
> > Working for me.
> > Regards*,*
> > *Mohammad Kathawala* | Sr. Technical Consultant
> > *HotWax Commerce* by *HotWax Systems*
> > Plot no. 80, Scheme no. 78 Part 2, Near Brilliant Convention Center,
> > Indore,
> > M.P 452010
> > Cell phone: 7772858789
> >
> > HotWax Systems recently received 8 mentions in *The Gartner Digital
> > Commerce Vendor Guide, 2016 *by Gartner, Inc., the world's leading IT
> > research and advisory company. Learn more about our research here
> > <
> >
> https://www.gartner.com/technology/media-products/newsletters/HotWax/1-2UVLP6M/index.html
> > >
> > .
> >
> >
> > On Sat, Sep 22, 2018 at 2:19 PM Rishi Solanki 
> > wrote:
> >
> > > Hello Team,
> > > Browse to Webtools >> Logging >> Service Log on demo trunk returns NPE.
> > >
> > > Quick Reference:
> > > https://demo-trunk.ofbiz.apache.org/webtools/control/ServiceLog
> > >
> > > Not able to replicate it on local setup, anyone face the same issue
> then
> > > please revert back. Will log Jira Ticket for that.
> > >
> > > Regards,
> > > --
> > > Rishi Solanki
> > > Sr Manager, Enterprise Software Development
> > > HotWax Systems Pvt. Ltd.
> > > Direct: +91-9893287847
> > > http://www.hotwaxsystems.com
> > > www.hotwax.co
> > >
> >
>

Re: Shipping data duplicated

[Rishi Solanki]

Jacques,
Thanks for more insights.
IMO, we should rename the files as you suggested and also add some
description in the file so that we won't confuse by this in future. And
also we should keep the duplicate data as well, because when ofbizsetup app
set the data for store I assume we don't load the demo data.
About the ofbizsetup app uses, right now I could not think of it. But will
get back on it soon with details and get the inputs from community. For now
my understanding is to rename the setup files and let duplicate data exists
in the setup files.
Let me know if we can proceed with the above plan then will rename the
files and do the needful changes in the setup code. On ofbizsetup app we
will discuss once I come with more details.

Thanks!

--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847
http://www.hotwaxsystems.com
www.hotwax.co


On Sat, Sep 22, 2018 at 2:10 PM Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:

> Hi Rishi,
>
> Thanks for your feedback.
>
> Looking closely at history, ShippingData.xml was put in 9 years ago with
> http://svn.apache.org/viewvc?view=revision=816083, so for the
> ofbizsetup app, which is a bit weirdo in OFBiz.
>
> Then was modified with OFBIZ-5890 and OFBIZ-7673
>
> Rest inline...
>
>
> Le 22/09/2018 à 09:27, Rishi Solanki a écrit :
> > Hi Jacques,
> > I dig into it today and found that the data exists in the
> > "applications/datamodel/data/demo/OrderDemoData.xml" was moved from
> > "applications/order/data/DemoShipping.xml"
> > Also the data claimed as duplicate in
> > "applications/commonext/data/ShippingData.xml" is similar but not exact
> > matches with OrderDemoData.xml.
> Yes, I was only speaking about "Shipping data", ie:
> ShipmentMethodType
> CarrierShipmentMethod
> QuantityBreak
> ShipmentBoxType
>
> Those are real duplicate
> > The ShippingData.xml  file has no entry in
> > any ofbiz-component.xml.
> Indeed, they are only used by the setup app. In SetupEvents.xml there is
>
> 
>  value="${sys:getProperty('ofbiz.home')}/applications/commonext/data/ShippingData.xml"/>
>
> Reading that, now I think we should not only keep the "Shipping data" in
> ShippingData.xml but also the file. I would rather rename this file and
> other ofbizsetup related files (at least data files) with a Setup prefix
> to clearly signal they are  part of this app.
> But I also wonder if the ofbizsetup app is still alive, maintained and
> used by users. Last time I tried I crossed issues (not biggie IIRW). I
> found
> this https://issues.apache.org/jira/issues/?filter=12344840
>
> What do you think?
>
> Jacques
> >
> > So here we can decide weather we should keep that data in
> ShippingData.xml
> > (if someone introduce the file intentionally) or we can remove it from
> > trunk. In case no objection I would like to remove it as most data is
> > duplicate.
> >
> > --
> > Rishi Solanki
> > Sr Manager, Enterprise Software Development
> > HotWax Systems Pvt. Ltd.
> > Direct: +91-9893287847
> > http://www.hotwaxsystems.com
> > www.hotwax.co
> >
> >
> > On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
> > jacques.le.r...@les7arts.com> wrote:
> >
> >> Thanks Rishi!
> >>
> >> Jacques
> >>
> >>
> >> Le 17/08/2018 à 15:19, Rishi Solanki a écrit :
> >>> This should be part of effort when we were moving all the seed, ext,
> demo
> >>> data to datamodel component. I see there is no entry to load the
> >>> ShippingData.xml.
> >>> I will check this in next week and fix the duplicate data exists in the
> >>> system, or share the the reason of having this.
> >>>
> >>> Probably, data moved but not removed from commonext. But I'll confirm
> and
> >>> get back.
> >>>
> >>> Thanks!
> >>>
> >>>
> >>>
> >>> Rishi Solanki
> >>> Sr Manager, Enterprise Software Development
> >>> HotWax Systems Pvt. Ltd.
> >>> Direct: +91-9893287847
> >>> http://www.hotwaxsystems.com
> >>> www.hotwax.co
> >>>
> >>> On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
> >>> jacques.le.r...@les7arts.com> wrote:
> >>>
>  Hi,
> 
>  Is there a reason why Shipping data are now duplicated in trunk at
> 
>    applications/commonext/data/ShippingData.xml
> 
>  and
> 
>    /applications/datamodel/data/demo/OrderDemoData.xml
> 
>  This is not the case with current stable
> 
>  Jacques
> 
> 
> >>
>
>

Re: Demo Trunk NPE

[Girish Vasmatkar]

Hi Rishi/Mohammad

This looks an intermittent issue as I could not reproduce it. However, in
webtools, under Fetch Logs, I tried searching for "NullPointer" in the
error.log file and it did return some instances of NullPointerException.

java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException
java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException
java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException
java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException
java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException
java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException
java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException
java.lang.IllegalArgumentException: Error running script at location
[component://webtools/groovyScripts/service/Services.groovy]:
java.lang.NullPointerException
Caused by: java.lang.NullPointerException

Best,
Girish Vasmatkar
HotWax Systems

On Sat, Sep 22, 2018 at 3:02 PM Mohammad Kathawala <
mohammad.kathaw...@hotwaxsystems.com> wrote:

> Working for me.
> Regards*,*
> *Mohammad Kathawala* | Sr. Technical Consultant
> *HotWax Commerce* by *HotWax Systems*
> Plot no. 80, Scheme no. 78 Part 2, Near Brilliant Convention Center,
> Indore,
> M.P 452010
> Cell phone: 7772858789
>
> HotWax Systems recently received 8 mentions in *The Gartner Digital
> Commerce Vendor Guide, 2016 *by Gartner, Inc., the world's leading IT
> research and advisory company. Learn more about our research here
> <
> https://www.gartner.com/technology/media-products/newsletters/HotWax/1-2UVLP6M/index.html
> >
> .
>
>
> On Sat, Sep 22, 2018 at 2:19 PM Rishi Solanki 
> wrote:
>
> > Hello Team,
> > Browse to Webtools >> Logging >> Service Log on demo trunk returns NPE.
> >
> > Quick Reference:
> > https://demo-trunk.ofbiz.apache.org/webtools/control/ServiceLog
> >
> > Not able to replicate it on local setup, anyone face the same issue then
> > please revert back. Will log Jira Ticket for that.
> >
> > Regards,
> > --
> > Rishi Solanki
> > Sr Manager, Enterprise Software Development
> > HotWax Systems Pvt. Ltd.
> > Direct: +91-9893287847
> > http://www.hotwaxsystems.com
> > www.hotwax.co
> >
>

Re: svn commit: r1841664 - /ofbiz/branches/release16.11/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilValidate.java

[Mathieu Lirzin]

Hello Suraj,

sur...@apache.org writes:

> Author: surajk
> Date: Sat Sep 22 09:26:34 2018
> New Revision: 1841664
>
> URL: http://svn.apache.org/viewvc?rev=1841664=rev
> Log:
> Fixed: Update regex used for validating credit cards.
> (OFBIZ-9761)
> Thanks Amit Gadaley for providing a patch for 16.11
>
> Modified:
> 
> ofbiz/branches/release16.11/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilValidate.java
>
> Modified: 
> ofbiz/branches/release16.11/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilValidate.java
> URL: 
> http://svn.apache.org/viewvc/ofbiz/branches/release16.11/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilValidate.java?rev=1841664=1841663=1841664=diff
> ==
> --- 
> ofbiz/branches/release16.11/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilValidate.java
>  (original)
> +++ 
> ofbiz/branches/release16.11/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilValidate.java
>  Sat Sep 22 09:26:34 2018
> @@ -1042,14 +1042,14 @@ public final class UtilValidate {
>  
>  /** Checks to see if the cc number is a valid Master Card number
>   *
> - * @param cc a string representing a credit card number; Sample number: 
> 5500   0004(16 digits)
> + * @param cc a string representing a credit card number; MasterCard 
> numbers either start with the numbers 51 through 55 or with the numbers 2221 
> through 2720. All have 16 digits; Sample number: 5500   0004(16 
> digits)
>   * @return true, if the credit card number is a valid MasterCard  
> number, false otherwise
>   */

Next time, please follow OFBiz standards by not adding lines longer than
120 Characters.

Thanks.

-- 
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761  070D 0ADE E100 9460 4D37

Re: Demo Trunk NPE

[Mohammad Kathawala]

Working for me.
Regards*,*
*Mohammad Kathawala* | Sr. Technical Consultant
*HotWax Commerce* by *HotWax Systems*
Plot no. 80, Scheme no. 78 Part 2, Near Brilliant Convention Center, Indore,
M.P 452010
Cell phone: 7772858789

HotWax Systems recently received 8 mentions in *The Gartner Digital
Commerce Vendor Guide, 2016 *by Gartner, Inc., the world's leading IT
research and advisory company. Learn more about our research here

.


On Sat, Sep 22, 2018 at 2:19 PM Rishi Solanki 
wrote:

> Hello Team,
> Browse to Webtools >> Logging >> Service Log on demo trunk returns NPE.
>
> Quick Reference:
> https://demo-trunk.ofbiz.apache.org/webtools/control/ServiceLog
>
> Not able to replicate it on local setup, anyone face the same issue then
> please revert back. Will log Jira Ticket for that.
>
> Regards,
> --
> Rishi Solanki
> Sr Manager, Enterprise Software Development
> HotWax Systems Pvt. Ltd.
> Direct: +91-9893287847
> http://www.hotwaxsystems.com
> www.hotwax.co
>

Demo Trunk NPE

[Rishi Solanki]

Hello Team,
Browse to Webtools >> Logging >> Service Log on demo trunk returns NPE.

Quick Reference:
https://demo-trunk.ofbiz.apache.org/webtools/control/ServiceLog

Not able to replicate it on local setup, anyone face the same issue then
please revert back. Will log Jira Ticket for that.

Regards,
--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847
http://www.hotwaxsystems.com
www.hotwax.co

Re: Shipping data duplicated

[Jacques Le Roux]

Hi Rishi,

Thanks for your feedback.

Looking closely at history, ShippingData.xml was put in 9 years ago with http://svn.apache.org/viewvc?view=revision=816083, so for the 
ofbizsetup app, which is a bit weirdo in OFBiz.


Then was modified with OFBIZ-5890 and OFBIZ-7673

Rest inline...


Le 22/09/2018 à 09:27, Rishi Solanki a écrit :

Hi Jacques,
I dig into it today and found that the data exists in the
"applications/datamodel/data/demo/OrderDemoData.xml" was moved from
"applications/order/data/DemoShipping.xml"
Also the data claimed as duplicate in
"applications/commonext/data/ShippingData.xml" is similar but not exact
matches with OrderDemoData.xml.

Yes, I was only speaking about "Shipping data", ie:
ShipmentMethodType
CarrierShipmentMethod
QuantityBreak
ShipmentBoxType

Those are real duplicate

The ShippingData.xml  file has no entry in
any ofbiz-component.xml.

Indeed, they are only used by the setup app. In SetupEvents.xml there is

   
   

Reading that, now I think we should not only keep the "Shipping data" in  ShippingData.xml but also the file. I would rather rename this file and 
other ofbizsetup related files (at least data files) with a Setup prefix to clearly signal they are  part of this app.
But I also wonder if the ofbizsetup app is still alive, maintained and used by users. Last time I tried I crossed issues (not biggie IIRW). I found 
this https://issues.apache.org/jira/issues/?filter=12344840


What do you think?

Jacques


So here we can decide weather we should keep that data in ShippingData.xml
(if someone introduce the file intentionally) or we can remove it from
trunk. In case no objection I would like to remove it as most data is
duplicate.

--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847
http://www.hotwaxsystems.com
www.hotwax.co


On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:


Thanks Rishi!

Jacques


Le 17/08/2018 à 15:19, Rishi Solanki a écrit :

This should be part of effort when we were moving all the seed, ext, demo
data to datamodel component. I see there is no entry to load the
ShippingData.xml.
I will check this in next week and fix the duplicate data exists in the
system, or share the the reason of having this.

Probably, data moved but not removed from commonext. But I'll confirm and
get back.

Thanks!



Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847
http://www.hotwaxsystems.com
www.hotwax.co

On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:


Hi,

Is there a reason why Shipping data are now duplicated in trunk at

  applications/commonext/data/ShippingData.xml

and

  /applications/datamodel/data/demo/OrderDemoData.xml

This is not the case with current stable

Jacques

Re: Shipping data duplicated

[Rishi Solanki]

Hi Jacques,
I dig into it today and found that the data exists in the
"applications/datamodel/data/demo/OrderDemoData.xml" was moved from
"applications/order/data/DemoShipping.xml"
Also the data claimed as duplicate in
"applications/commonext/data/ShippingData.xml" is similar but not exact
matches with OrderDemoData.xml. The ShippingData.xml  file has no entry in
any ofbiz-component.xml.

So here we can decide weather we should keep that data in ShippingData.xml
(if someone introduce the file intentionally) or we can remove it from
trunk. In case no objection I would like to remove it as most data is
duplicate.

--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847
http://www.hotwaxsystems.com
www.hotwax.co


On Sat, Aug 18, 2018 at 1:17 AM Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:

> Thanks Rishi!
>
> Jacques
>
>
> Le 17/08/2018 à 15:19, Rishi Solanki a écrit :
> > This should be part of effort when we were moving all the seed, ext, demo
> > data to datamodel component. I see there is no entry to load the
> > ShippingData.xml.
> > I will check this in next week and fix the duplicate data exists in the
> > system, or share the the reason of having this.
> >
> > Probably, data moved but not removed from commonext. But I'll confirm and
> > get back.
> >
> > Thanks!
> >
> >
> >
> > Rishi Solanki
> > Sr Manager, Enterprise Software Development
> > HotWax Systems Pvt. Ltd.
> > Direct: +91-9893287847
> > http://www.hotwaxsystems.com
> > www.hotwax.co
> >
> > On Wed, Aug 15, 2018 at 10:10 PM, Jacques Le Roux <
> > jacques.le.r...@les7arts.com> wrote:
> >
> >> Hi,
> >>
> >> Is there a reason why Shipping data are now duplicated in trunk at
> >>
> >>  applications/commonext/data/ShippingData.xml
> >>
> >> and
> >>
> >>  /applications/datamodel/data/demo/OrderDemoData.xml
> >>
> >> This is not the case with current stable
> >>
> >> Jacques
> >>
> >>
>
>

Issue: Additional shipping charge editable on shipment even after shipment is packed and invoice is generated

[Mohammad Kathawala]

Hello Devs,

I need to confirm if the following scenario is valid or not:
At the time of packing an order, an additional shipping charge can be added
from the packing screen which is stored in the shipment entity.
When package/s are completed, the shipment is marked as packed and an
invoice is generated for the order in ready for posting status. Also, an
invoice line item is created from the additional shipping charge added in
the shipment. Now, since the invoice is in ready for posting status, it
cannot be edited. But, the additional shipping charge on the shipment is
still editable. Is this a valid scenario?

My second concern is if someone has entered wrong shipping charge at the
time of packing then how can it be corrected since the invoice gets
generated as soon as shipment is packed.

Thanks.
Regards*,*
*Mohammad Kathawala* | Sr. Technical Consultant
*HotWax Commerce* by *HotWax Systems*
Plot no. 80, Scheme no. 78 Part 2, Near Brilliant Convention Center, Indore,
M.P 452010
Cell phone: 7772858789

HotWax Systems recently received 8 mentions in *The Gartner Digital
Commerce Vendor Guide, 2016 *by Gartner, Inc., the world's leading IT
research and advisory company. Learn more about our research here

.

Re: Inventory Cycle Count feature gap

[Yashwant Dhakad]

Hello All,
I have created a ticket 
for this and provided all the design notes.

Thanks & Regards
--
Yashwant Dhakad


On Mon, Jul 30, 2018 at 5:03 PM Swapnil Shah 
wrote:

> One more scenario to consider while weighing in the design is that all the
> categories' items are not necessarily required to be counted in the
> facility. And at times these categories have dedicated locations for
> stocking. For example, supply items or consumables.
>
> It would be nice to make it configurable such that only eligible
> categories'
> locations/items are listed for counting by default.
>
> Thanks & Regards,
> Swapnil
>
> -Original Message-
> From: Yashwant Dhakad 
> Sent: Saturday, July 28, 2018 5:07 PM
> To: dev@ofbiz.apache.org
> Cc: Yashwant Dhakad 
> Subject: Inventory Cycle Count feature gap
>
> Hello All,
> Currently, we don't have any user interface to do cycle count for the
> warehouse, Where user can create or manage cycle count for different
> locations within any facility. Authorized users to approve or reject the
> count and record variance. Also, the recorded variance against
> counted/scanned locations can be made available in the form of the report
> on
> a daily basis. Based on this I am proposing this feature where we can
> record
> and manage the cycle count through different sessions in following
> steps:
>
>1. Create Cycle Count session: User can create a new cycle count session
>by scanning any location from the facility and system would list all the
>inventory items or product ids. The user can physically count or scan
> the
>inventory/product and update the physical count. Apart from this, user
> can
>also be allowed to manually add any not listed inventory item or product
> in
>the same session. Once record count is completed user can submit the
>cycle count session for review. As soon as the session is created the
> added
>locations can be locked from honoring any inward or outward inventory
>movement or transactions through them.
>2. Find Counting session: On this screen, User can view all the existing
>sessions and further filter by session id, facility, location and
> status.
>3. Review Cycle Count session:  On this screen, any authorized user can
>review the cycle count for open sessions and accept or reject the
> recorded
>count, once done system would create the
>necessary variance to corresponding inventory item or product and update
>the ATP/QOH accordingly.
>4. Variance Report screen: On this screen, the user can view the actual
>variance based on facility, location, product/inventory and date.
>5. Later on if required we can also schedule the submitted location for
>next cycle count by setting next counting date. This would allow to user
>see all the pending location on any given date where cycle count is due.
>And can start new sessions for the same.
>
> Please share your thoughts and we make the necessary changes in the
> workflow
> based on that feedback.
>
> Thanks & Regards
> --
> Yashwant Dhakad
>

Re: Date filtered queries shouldn't be cached

[Suraj Khurana]

+1

I have created a Jira 
for this as per suggestion from Scott.

--
Thanks and Regards,
Suraj Khurana
Omnichannel OMS Technical Expert
HotWax Systems





On Tue, Apr 11, 2017 at 1:31 PM, Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:

> Hi Scott,
>
> +1 on both your propositions below
>
> Jacques
>
>
>
> Le 07/04/2017 à 06:04, Scott Gray a écrit :
>
>> Hi Taher,
>>
>> Thanks for taking a look.
>>
>> It's not about monitoring EntityCondition creations, but about inspecting
>> the EntityCondition supplied with the cache request e.g.
>> LocalCache.put(...) to determine if it contains an "uncachable" condition.
>>
>> Regarding looking for "fromDate" vs. something more generic, I'm not sure
>> I
>> follow.  from/thruDates in OFBiz are an established pattern for
>> determining
>> whether a record is active or not.  Other timestamp fields serve other
>> purposes and the issues I've described don't apply to them.  Yes, I would
>> be relying upon the convention of naming a fromDate as "fromDate"
>> (actually
>> more like fieldName.toLowerCase().contains("fromdate")) but that's a
>> pretty
>> safe assumption to make isn't it?
>>
>> Back to my original question, it occurs to me that it might be possible a
>> developer would want to retrieve all rows that would be valid on a
>> specific
>> date other than "right now".  I can't think of a concrete scenario in
>> which
>> that might be useful but it does give me pause and perhaps it might be
>> safer to err on the side of caution and only look for
>> EntityDateFilterConditions which are 100% guaranteed to fit the scenario
>> of
>> results we don't want in the cache.
>>
>> Regards
>> Scott
>>
>> On 7 April 2017 at 00:34, Taher Alkhateeb 
>> wrote:
>>
>> Hi Scott,
>>>
>>> Maybe I am not understanding your solution correctly, so I summarize
>>> what I
>>> understood:
>>>
>>> - Monitor all EntityCondition creations from
>>> EntityDateFilterCondition,makeCondition
>>> or EntityUtil.filterByDate
>>> - If the query has a field called "fromDate" then log a warning and
>>> disable
>>> caching
>>>
>>> If my above understanding is correct, then isn't that a bit strange? I
>>> mean
>>> it seems to be quite a specific workaround as opposed to a generic
>>> solution. Shouldn't we instead have something that works for all dates
>>> instead of just specific dates with the name "fromDate" when compared
>>> with
>>> current date / time?
>>>
>>> On Thu, Apr 6, 2017 at 2:10 AM, Scott Gray >> >
>>> wrote:
>>>
>>> In the current project I'm working on I see a lot of developers making

>>> the
>>>
 mistake of caching date filtered queries.

 Why shouldn't you cache date filtered queries?
 1. If you're filtering by the current moment in time then the results
 are
 irrelevant within a few moments of them being retrieved.  Some records

>>> may
>>>
 be due to expire and others (that were filtered out) may be about to

>>> become
>>>
 active.
 2. Despite what I say in #1, there's a bug in the equals method of
 EntityDateFilterCondition where all instances appear equal so the cached
 result is never replaced until it is expired from the cache. So expired
 records remain and future dated records never appear until the TTL is
 reached.

 The correct approach is to not have a date filter in the query, cache
 the
 result and then filter the result in-memory using
 EntityUtil.filterByDate(). An alternative approach is to only filter by
 thruDate in the query since expired records will never become active,
 and
 then filter by fromDate after you have the result.  We have no utility
 tools in place for the latter approach though.

 I'm writing all this because I've written a checker for the entity list

>>> and
>>>
 object caches that inspect the query conditions for fromDate conditions

>>> and
>>>
 if found, log a warning and refuse to cache.  I'd like to add this
 safety
 feature to the OFBiz trunk but I'm wondering if anyone can think of a

>>> valid
>>>
 reason to cache fromDate filtered records? I haven't been able to think

>>> of
>>>
 any but that doesn't necessarily mean there aren't any use cases.

 Thanks
 Scott


>

Re: Preparing the new release 16.11.05

[Rishi Solanki]

+1.

--
Rishi Solanki
Sr Manager, Enterprise Software Development
HotWax Systems Pvt. Ltd.
Direct: +91-9893287847
http://www.hotwaxsystems.com
www.hotwax.co


On Fri, Sep 21, 2018 at 7:52 PM Jacopo Cappellato <
jacopo.cappell...@hotwaxsystems.com> wrote:

> No feedback so far; I will proceed with the release preparation.
>
> Jacopo
>
> On Mon, Sep 17, 2018 at 11:09 AM Jacopo Cappellato <
> jacopo.cappell...@hotwaxsystems.com> wrote:
>
> > I am ready to prepare the release files and start a vote but before I do
> I
> > would like to double check about OFBIZ-4361: if people think that it
> should
> > be considered blocker then we could disable the link/feature in the
> release
> > branch and proceed with the release process; when we will have a
> > stable/tested/agreed upon refactoring of this "reset password" feature we
> > could re-enable the link.
> >
> > Jacopo
> >
> > On Fri, Sep 14, 2018 at 11:48 AM Taher Alkhateeb <
> > slidingfilame...@gmail.com> wrote:
> >
> >> I don't think this issue is a blocker for a new release, nor does it
> >> necessarily warrant a release just for that. It can just be part of a
> >> batch
> >> as usual.
> >>
> >> On Fri, Sep 14, 2018, 12:09 PM Jacopo Cappellato <
> >> jacopo.cappell...@hotwaxsystems.com> wrote:
> >>
> >> > On Mon, Sep 10, 2018 at 2:47 PM Pierre Smits 
> >> > wrote:
> >> >
> >> > > Should OFBIZ-4361 not get resolved for this?
> >> >
> >> >
> >> > It makes sense, thank you.
> >> > I have posted a comment in the ticket and hopefully we will come up
> >> with a
> >> > quick resolution.
> >> > Should we wait a few more days to see if we can commit and test that
> >> work
> >> > before we start the release voting process? However I wouldn't delay
> the
> >> > release preparation more than a few days and I would rather issue this
> >> > release and then issue another one in a few weeks.
> >> >
> >> > Regards,
> >> >
> >> > Jacopo
> >> >
> >>
> >
>