Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AD_Column.IsMandatory shall be overwritten by MandatoryLogic #607

Closed
metas-ts opened this issue Sep 21, 2017 · 3 comments
Closed

AD_Column.IsMandatory shall be overwritten by MandatoryLogic #607

metas-ts opened this issue Sep 21, 2017 · 3 comments

Comments

@metas-ts
Copy link
Member

Is this a bug or feature request?

FR

What is the current behavior?

In the webui AD_Column.MandatoryLogic is only evaluated if AD_Column.IsMandatory='N'.

Which are the steps to reproduce?

i saw it with C_FlatrateCondidtions.UOMType..

What is the expected or desired behavior?

if AD_Column.MandatoryLogicis specified, go with it. Use AD_Column.IsMandatory='N' as "fallback" in case no logic is specified

@metas-ts metas-ts self-assigned this Sep 21, 2017
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Sep 21, 2017
also clean up some unrelated files i found lying around

AD_Column.IsMandatory shall be overwritten by MandatoryLogic
metasfresh/metasfresh-webui-api-legacy#607
metas-ts added a commit that referenced this issue Sep 21, 2017
also: 
* renamed a method in DocumentField to be more informative
* minor stuff

AD_Column.IsMandatory shall be overwritten by MandatoryLogic
#607
@metas-ts metas-ts removed their assignment Sep 21, 2017
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Sep 24, 2017
[#608](metasfresh/metasfresh-webui-api-legacy#608) CU
Quantity in Excel Export from HU Editor is wrong
[#2477](#2477)
Subscription Import Window Adjustments
[#588](metasfresh/metasfresh-webui-api-legacy#588)
Cloning Feature: Error when cloning BOM
[#2541](#2541) Allow
filtering by product group in window product
[#2538](#2538) Remove
legacy LDAP code
[#576](metasfresh/metasfresh-webui-api-legacy#576)
Implement batch entry for Sales/Purchase Invoice documents
[#2484](#2484) allow
using partner product numbers and name for entering order line
[#2502](#2502) Allow
multiple shipment schedules per orderline
[#2395](#2395) Drop
C_Flatrate_Conditions.IsNewTermCreatesOrder together with java code
[#2529](#2529) Import
System users
[#607](metasfresh/metasfresh-webui-api-legacy#607)
AD_Column.IsMandatory shall be overwritten by MandatoryLogic
[#2514](#2514) Support
importing multiline fields
[#603](metasfresh/metasfresh-webui-api-legacy#603)
Handle incorrect AD_Window_ID in URL
[#1167](metasfresh/metasfresh-webui-frontend-legacy#1167)
Edit Fields in Main Grid View
[#2146](#2146) Implement
Generic Multi Tags / labels for records
[#2501](#2501) Import
Requests
[#2510](#2510) Prevent
users from creating duplicate main prices
[#2344](#2344) Support
version 02 CAMT54 file
[#1197](metasfresh/metasfresh-webui-frontend-legacy#1197)
frontend: export view to excel
[#2482](#2482) Allow
disabling of model interceptors on the fly
[#1194](metasfresh/metasfresh-webui-frontend-legacy#1194)
Use API to fetch the locale
[#591](metasfresh/metasfresh-webui-api-legacy#591)
Filter by warehouse when selecting HUs to issue
[#593](metasfresh/metasfresh-webui-api-legacy#593)
Remove Clone action if not supported
[#2492](#2492) Process
C_Dunning_Candidate_Process fails
[#491](metasfresh/metasfresh-webui-api-legacy#491)
Wrong widget Type for Date-Range in Process Parameter

me-45
@teosarca teosarca self-assigned this Sep 25, 2017
@teosarca teosarca reopened this Sep 25, 2017
metas-ts pushed a commit to metasfresh/metasfresh that referenced this issue Sep 25, 2017
metas-ts pushed a commit to metasfresh/metasfresh that referenced this issue Sep 25, 2017
metas-ts pushed a commit that referenced this issue Sep 25, 2017
#607
(cherry picked from commit d1c0fb0)
@metas-ts
Copy link
Member Author

cherry-picked @teosarca's fix to the release branch

@metas-dh
Copy link
Member

Results of IT1
tested in mf15

  • new pricelist: pricing system is marked as mandatory field, so no porblems with setting a name for the pricelist / changing the currency (was not possible during last e2e bc of this problem): OK
  • will test once more in next e2e

@metas-dh
Copy link
Member

tested in mf15releasetest during e2e, working fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants