Info Window - parameters - field Mandatory Logic missing

Description

we found Mandatory logic to be implemented when discussed about https://idempiere.atlassian.net/browse/IDEMPIERE-1963 and https://idempiere.atlassian.net/browse/IDEMPIERE-1972

reason: we met in - we need to denied for user make empty e.g. document type dropdown in this case.

Environment

actual trunk

Activity

Show:
Hiep Lq
May 30, 2014, 3:45 PM

I finish will mandatoryness as advice. maybe i misunderstand at something

below is my logic:
+ add field mandatory to infoColumn
+ display field mandatory normal in parameter panel (without red color and *)
+ when click query button, check mandatory, if not input, display in red color and not query

please confirm it

Hiep Lq
June 2, 2014, 4:33 AM

add logic check mandatory when user enter at parameter field

Carlos Ruiz
June 4, 2014, 11:49 PM

, I think it could be better to implement MandatoryLogic instead of IsMandatory (similar to how the mandatory logic is implemented on process parameters).

Also the display logic of the new field must be visible just if is a query criteria (similar to the display logic I added to the default logic on

I was not able to integrate cleanly the IDEMPIERE-1973-para-field.patch - but integrating manually and after deleting a variable that didn't exist, in my tests noticed:

  • the label of mandatory query criteria is not red until you push the refresh button, I think better must be red when starting the window

  • the label then turns correctly to black if the criteria is filled

Regards,

Carlos Ruiz

Hiep Lq
June 5, 2014, 2:35 AM

many thanks for review.
will change to logic mandatory.

about comment: the label of mandatory query criteria is not red until you push the refresh button, I think better must be red when starting the window
it's by same issue in https://idempiere.atlassian.net/browse/IDEMPIERE-1979

other solution is: still display mandatory field in red, but not check in case auto requery.

Hiep Lq
June 11, 2014, 3:16 AM

this patch is depend to 1979, please apply 1979

Assignee

Unassigned

Reporter

Norbert Bede

Labels

Tested By

None

Components

Affects versions

Due date

2014/05/28

Priority

Major
Configure