ofbiz

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

Changeset 1863485 is being indexed.

Improved: Inline ‘UtilGenerics#checkCollectionCast’

(OFBIZ-11141)

"Applied fix from trunk framework for revision: 1835871"

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

r1835871 | surajk | 2018-07-14 07:35:44 +0200 (sam. 14 juil. 2018) | 10 lignes

Improved: All existing party contact mechs should be deleted before creating new one.

(OFBIZ-10396)

Currently, everywhere in code base, before creating new party contact mech record, we use

deletePartyContactMechPurposeIfExists service to delete old records. But, problem is

<first-from-list is used everywhere which only deletes first fetched record.

All previous records should also gets deleted before creating new party contact mech purpose for a specific purpose type.

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

?\026

"Applied fix from trunk for revision: 1835871"

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

r1835871 | surajk | 2018-07-14 07:35:44 +0200 (sam. 14 juil. 2018) | 10 lignes

Improved: All existing party contact mechs should be deleted before creating new one.

(OFBIZ-10396)

Currently, everywhere in code base, before creating new party contact mech record, we use

deletePartyContactMechPurposeIfExists service to delete old records. But, problem is

<first-from-list is used everywhere which only deletes first fetched record.

All previous records should also gets deleted before creating new party contact mech purpose for a specific purpose type.

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

Improved: Remove useless ‘UtilObject#compareToHelper’

(OFBIZ-11140)

Improved: Remove useless ‘UtilObject#getBytes’

(OFBIZ-11140)

Improved: Remove useless ‘UtilObject#equalsHelper’

(OFBIZ-11140)

Improved: Remove ‘UtilMisc#toList’

(OFBIZ-11140)

Improved: Refactor ‘Makeparamlistwithsuffix’

(OFBIZ-11138)

Implemented: Add unit tests for ‘UtilHttp#makeParamListWithSuffix’

(OFBIZ-11138)

Adapt implementation to facilitate mocking.

Improved: Refactor ‘UtilHttp#makeParamValueFromComposite’

(OFBIZ-11138)

Use ‘LocalDateTime’ instead of ‘Calendar’ and

‘HttpServletRequest#getParameterMap’ instead of

‘HttpServletRequest#getParameterNames’.

The tests and callers has been adapted to remove the ‘locale’ arguments which

made sense only for ‘Calendar’.

Implemented: Add unit tests for ‘UtilHttp#makeParamValueFromComposite’

(OFBIZ-11138)

The implementation has been adapted to facilitate mocking.

Improved: Use ‘HttpServletRequest#getParameterMap’ when possible

It is better to avoid using obsolete ‘Enumeration’ class when possible

so the calls to ‘HttpServletRequest#getParameterNames’ has been

replaced by ‘HttpServletRequest#getParameterMap’.

Improved: Remove useless ‘UtilHttp#parametersToAttributes’

(OFBIZ-11138)

This method was so trivial that it doesn't make sense to provide it.

Improved: Remove unused ‘UtilHttp#checkURLforSpiders’

(OFBIZ-11138)

It was not used anywhere in the framework or the official plugins.

Improved: Use a predicate in ‘UtilHttp#getParameterMap’

(OFBIZ-11138)

Improved: Use a predicate in ‘UtilHttp#getPathInfoOnlyParameterMap’

(OFBIZ-11138)

Improved: Refactor ‘UtilHttp#getParameterMap’

(OFBIZ-11138)

Implemented: Add unit tests for ‘UtilHttp#getParameterMap’

(OFBIZ-11138)

Adapt slightly the implementation to make mocking easier.

Improved: Remove unused ‘UtilHttp#getParameterMap’ overload

(OFBIZ-11138)

Improved: Refactor ‘UtilHttp#getPathInfoOnlyParameterMap’

(OFBIZ-11138)

Implemented: Add unit tests for ‘UtilHttp#getPathInfoOnlyParameterMap’

(OFBIZ-11138)

Improved: Inline one ‘UtilHttp#getPathInfoOnlyParameterMap’ overload

(OFBIZ-11138)

‘getPathInfoOnlyParameterMap(HttpServletRequest, Set, Boolean)’ was

used only once and was not adding much to

‘getPathInfoOnlyParameterMap(String, Set, Boolean)’.

Improved: Do not use ‘UtilMisc#toList’

(OFBIZ-11140)

Improved: Upgrade Hamcrest library to version 2.1

(OFBIZ-11139)

Improved: Remove unnecessary dependency on ‘junit-dep’ artifact

(OFBIZ-11139)

Prior to Junit 4.11, Junit was distributed in two forms ‘junit’ and

‘junit-dep’ where the first was embedding the hamcrest matchers the

second was defining a dependency to it which is more desirable in the

context of package managers like Maven or Gradle. Starting with Junit

4.11 only the second form is distributed which makes ‘junit-dep’

obsolete.

Improved: Move ‘AdminServer’ inside a container

(OFBIZ-11136)

‘AdminServer’ provides a portable way to manage life-cycle of the OFBiz

process remotely by allowing administrator to check its running status

or shutting it down.

Previously the ‘AdminServer’ class was a special thread opening a

socket and launched at startup. However since this class is about

managing some run-time resources with a life-cycle, it matches perfectly

the container abstraction.

A benefit of making ‘AdminServer’ a container is that the startup

process is now simpler and more uniform.

Administrators can now prevent remote shutdown of OFBiz for security

reasons by removing the container declaration. Additionally They can

delegate the process management job to the init process (PID 0) of the

hosting system like Systemd [1] by replacing this container with

another one.

[1] https://www.freedesktop.org/software/systemd/man/systemd-notify.html

Improved: Pass directly the loaders to ‘loadContainersFromConfigurations’

(OFBIZ-11100)

Since only the loaders are used from the configuration, it is better

to pass those directly as a parameter to the

‘loadContainersFromConfigurations’ method instead of the full

configuration object.

Improved: Access configurations inside ‘loadContainersFromConfigurations’

(OFBIZ-11100)

This avoids passing an extra argument to

‘loadContainersFromConfigurations’.

Improved: Inline ‘filterContainersHavingMatchingLoaders’

(OFBIZ-11100)

The filtering of matching loaders is now done directly inside the

‘loadContainersFromConfigurations’ method.

Improved: Deprecate ‘ContainerConfig#getConfiguration(String, String)’

(OFBIZ-11100)

Since there is no global container configuration file anymore this

method should not be used. It has been superseded by an overload

which do not require to pass the file name of the configuration file.