ofbiz

Checkout Tools
  • last updated 6 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates

Changeset 1868000 is being indexed.

Improved: no functional change

As reported by Ethan Vos on user ML, sometimes the alias between wget and

Invoke-WebRequest is not present on Windows 7, better keep it simple!

Also removes the comment:

Anyway I believe this should be only used in dev environment

which is no longer true

"Applied fix from trunk for revision: 1867998"

------------------------------------------------------------------------

r1867998 | jleroux | 2019-10-05 10:05:54 +0200 (sam. 05 oct. 2019) | 8 lignes

Improved: no functional change

As reported by Ethan Vos on user ML, sometimes the alias between wget and

Invoke-WebRequest is not present on Windows 7, better keep it simple!

Also removes the comment:

Anyway I believe this should be only used in dev environment

which is no longer true

------------------------------------------------------------------------

Improved: no functional change

As reported by Ethan Vos on user ML, sometimes the alias between wget and

Invoke-WebRequest is not present on Windows 7, better keep it simple!

Also removes the comment:

Anyway I believe this should be only used in dev environment

which is no longer true

Improved: Remove unnecessary semicolon

Improved: Remove unused imports

Improved: Add missing generic types

Applied fix from trunk for revision: 1867972

===

Reverted: User is unable to update the review of product

(OFBIZ-10799)

Reverted rev 1867904

Thanks: Deepak Dixit for reporting the issue.

Applied fix from trunk for revision: 1867972

===

Reverted: User is unable to update the review of product

(OFBIZ-10799)

Reverted rev 1867904

Thanks: Deepak Dixit for reporting the issue.

Applied fix from trunk for revision: 1867972

===

Reverted: User is unable to update the review of product

(OFBIZ-10799)

Reverted rev 1867904

Thanks: Deepak Dixit for reporting the issue.

Reverted: User is unable to update the review of product

(OFBIZ-10799)

Reverted rev 1867904

Thanks: Deepak Dixit for reporting the issue.

Improved: Use website to generate links on email content

(OFBIZ-4361)

OFBiz contains a nice process to generate link through WebSite entity.

Unfortunately when you send an email, the standard service didn't propage the website

to the body content email, so we can't use it.

This improvement is needed on forgot password process to resolve the correct uri

and go back on good OFBiz component: where the user requested for a new password.

Improved: Use website to generate links on email content

(OFBIZ-4361)

OFBiz contains a nice process to generate link through WebSite entity.

Unfortunately when you send an email, the standard service didn't propage the website

to the body content email, so we can't use it.

This improvement is needed on forgot password process to resolve the correct uri

and go back on good OFBiz component: where the user requested for a new password.

Applied fix from trunk for revision: 1867937

===

'At least one phone number is required below' message should be uneditable while creating new customer using partymgr

(OFBIZ-11113)

Thanks: Arpit Mor for report and Saurabh Dixit for the patch.

Applied fix from trunk for revision: 1867937

===

'At least one phone number is required below' message should be uneditable while creating new customer using partymgr

(OFBIZ-11113)

Thanks: Arpit Mor for report and Saurabh Dixit for the patch.

Applied fix from trunk for revision: 1867937

===

'At least one phone number is required below' message should be uneditable while creating new customer using partymgr

(OFBIZ-11113)

Thanks: Arpit Mor for report and Saurabh Dixit for the patch.

'At least one phone number is required below' message should be uneditable while creating new customer using partymgr

(OFBIZ-11113)

Thanks: Arpit Mor for report and Saurabh Dixit for the patch.

Improved: Unit test case for service - sendOrderCompleteNotification

(OFBIZ-8812)

As Suraj somehow mentioned at https://markmail.org/message/zlf5chufefekb5r2,

we can use the same ticket while converting the test to Groovy

Tests pass

Thanks: Avnindra Sharma for the initial patch (here converted to Groovy)

Applied fix from trunk for revision: 1867930

===

Fixed: Check run payment(A/P) transactions not getting available for reconciliation

(OFBIZ-10796)

Check run payment group transactions don't get available for Financial Account reconciliation. Though the financial account balance gets updated during the A/P check run process, the system must allow the user with an additional step of reconciling the transactions.

The cause of this is that the financial account transaction doesn't get created in the 'Created' status and has an 'Approved' status. An approved status financial transaction directly updates the financial account balance and thus don't appear in the list of the transactions to be reconciled.

Thanks: Chinmay Patidar for your contribution and Pierre Smits for the review.

Applied fix from trunk for revision: 1867930

===

Fixed: Check run payment(A/P) transactions not getting available for reconciliation

(OFBIZ-10796)

Check run payment group transactions don't get available for Financial Account reconciliation. Though the financial account balance gets updated during the A/P check run process, the system must allow the user with an additional step of reconciling the transactions.

The cause of this is that the financial account transaction doesn't get created in the 'Created' status and has an 'Approved' status. An approved status financial transaction directly updates the financial account balance and thus don't appear in the list of the transactions to be reconciled.

Thanks: Chinmay Patidar for your contribution and Pierre Smits for the review.

Applied fix from trunk for revision: 1867930

===

Fixed: Check run payment(A/P) transactions not getting available for reconciliation

(OFBIZ-10796)

Check run payment group transactions don't get available for Financial Account reconciliation. Though the financial account balance gets updated during the A/P check run process, the system must allow the user with an additional step of reconciling the transactions.

The cause of this is that the financial account transaction doesn't get created in the 'Created' status and has an 'Approved' status. An approved status financial transaction directly updates the financial account balance and thus don't appear in the list of the transactions to be reconciled.

Thanks: Chinmay Patidar for your contribution and Pierre Smits for the review.

Fixed: Check run payment(A/P) transactions not getting available for reconciliation

(OFBIZ-10796)

Check run payment group transactions don't get available for Financial Account reconciliation. Though the financial account balance gets updated during the A/P check run process, the system must allow the user with an additional step of reconciling the transactions.

The cause of this is that the financial account transaction doesn't get created in the 'Created' status and has an 'Approved' status. An approved status financial transaction directly updates the financial account balance and thus don't appear in the list of the transactions to be reconciled.

Thanks: Chinmay Patidar for your contribution and Pierre Smits for the review.

Improved: Unit test case for service - sendOrderPayRetryNotification

(OFBIZ-8813)

As Suraj somehow mentioned at https://markmail.org/message/zlf5chufefekb5r2,

we can use the same ticket while converting the test to Groov

Tests pass

Thanks: Avnindra Sharma for the initial patch (here converted to Groovy)

Improved: Unit test case for service - SendOrderBackorderNotification

(OFBIZ-8810)

As Suraj somehow mentioned at https://markmail.org/message/zlf5chufefekb5r2,

we can use the same ticket while converting the test to Groov

Tests pass

Thanks: Avnindra Sharma for the initial patch (here converted to Groovy)

Improved: Convert testSendOrderChangeNotification to Groovy

(OFBIZ-11233)

Tests pass

Improved: Unit test case for service - testSendOrderChangeNotification

(OFBIZ-8811)

Removes duplicated order-tests-data-load

Tests pass

Improved: Unit test case for service - testSendOrderChangeNotification

(OFBIZ-8811)

Fixes to right place

There were no reasons to add an OrderTest.xml file,

testSendOrderChangeNotification fits in OrderTests.xml

It was even wrong because of duplicated order-test test-case

Tests pass

Applied fix from trunk for revision: 1867910

===

Fixed: Sales By Store Report not working

(OFBIZ-11119)

Added Null check to avoid Exception.

Thanks: Chandan Khandelwal for your contribution.

Applied fix from trunk for revision: 1867910

===

Fixed: Sales By Store Report not working

(OFBIZ-11119)

Added Null check to avoid Exception.

Thanks: Chandan Khandelwal for your contribution.

Applied fix from trunk for revision: 1867910

===

Fixed: Sales By Store Report not working

(OFBIZ-11119)

Added Null check to avoid Exception.

Thanks: Chandan Khandelwal for your contribution.

Fixed: Sales By Store Report not working

(OFBIZ-11119)

Added Null check to avoid Exception.

Thanks: Chandan Khandelwal for your contribution.