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

Address missing in 'Adresse' for bpartner created on-the-fly #306

Closed
metas-dh opened this issue Apr 21, 2017 · 5 comments
Closed

Address missing in 'Adresse' for bpartner created on-the-fly #306

metas-dh opened this issue Apr 21, 2017 · 5 comments

Comments

@metas-dh
Copy link
Member

Is this a bug or feature request?

Bug

What is the current behavior?

If you create a bpartner on-the-fly, e.g. in purchase order, and set an address there, then this address is not displayed in Adresse, bpartner window, tab address, only in Anschrift

Which are the steps to reproduce?

  • open New Purchase Order
  • in bpartner field, write e.g jhug, then click on new bpartner
  • create a new bpartner on-the-fly, and also set an address there!
  • after creating the prder, check the bpartner window for this new bpartner
    => tab address: the address you set when creating the bpartner is displayed in field Anschrift, but not in Adresse

What is the expected or desired behavior?

The address should be displayed in the Adresse field too.

@teosarca
Copy link
Member

@metas-dh quick checked on my laptop. It looks ok imho?!

image

@teosarca
Copy link
Member

UPDATE: found the reason why the C_Location_ID is displayed empty:

  • the C_Location_ID display string is built from City
  • the City was not filled.

Suggestion:

  • make the city mandatory (don't see a reason to not be mandatory)
  • introduce C_Location.Address field which shall always contain the prebuilt location string

@teosarca
Copy link
Member

UPDATE:

  • marked C_Location.C_Country_ID as identifier (so it will be part of the lookup display string)

@teosarca teosarca added this to the 2017-17 milestone Apr 23, 2017
@metas-lc metas-lc self-assigned this Apr 24, 2017
@metas-lc
Copy link

IT

  1. go to purchase order and create a new one
  2. start writing a bpartner name that does not exists
  3. create a new bpartner on-the-fly, and also set an address there (e.g. aaa 1234 country: DE)
  4. finish the order and go to business partner window, see address tab
    => first field is filled with country name (DE) OK
  5. right click to Advanced Edit, select the address button, close it
    => notice that now on the address field is "aaa 1234" instead of the country's name OK?

@metas-mk
Copy link
Member

@metas-lc thanks for testing
about "right click to Advanced Edit, select the address button, close it"
I would leave it like that atm. We can take care of this minor case later.

metas-ts added a commit to metasfresh/metasfresh that referenced this issue Apr 28, 2017
[#1382](#1382) Make Packing Item editable in Manufacturing Order Header
[#323](metasfresh/metasfresh-webui-api-legacy#323) Manufacturing Issue/Receipt: receipt lines shall be displayed first
[#689](metasfresh/metasfresh-webui-frontend-legacy#689) Image widget is missing right border
[#326](metasfresh/metasfresh-webui-api-legacy#326) technical: Remove JSONProcessInstanceResult deprecated properties
[#680](metasfresh/metasfresh-webui-frontend-legacy#680) Navigation breadcrumb vs. Navigation Menu
[#329](metasfresh/metasfresh-webui-api-legacy#329) Issue HU 404 not found#
[#688](metasfresh/metasfresh-webui-frontend-legacy#688) Main page /dashboard is broken
[#667](metasfresh/metasfresh-webui-frontend-legacy#667) Change /rest/api/pattribute request message
[#319](metasfresh/metasfresh-webui-api-legacy#319) Manufacturing Issue/Receipt: show BOM line attributes if any
[#317](metasfresh/metasfresh-webui-api-legacy#317) Manufacturing Issue/Receipt: drop StatusInfo column
[#248](metasfresh/metasfresh-webui-frontend-legacy#248) Handle Password Fields in WebUI
[#678](metasfresh/metasfresh-webui-frontend-legacy#678) Get rid of deprecated static docNo input in header.
[#151](metasfresh/metasfresh-webui-api-legacy#151) grid view websocket notifications: fullyChanged shall not be set when changedIds is set
[#664](metasfresh/metasfresh-webui-frontend-legacy#664) process doesn't open directly
[#311](metasfresh/metasfresh-webui-api-legacy#311) Switching language does not translate document lookup values
[#310](metasfresh/metasfresh-webui-api-legacy#310) Switching language does not translate document references
[#306](metasfresh/metasfresh-webui-api-legacy#306) Address missing in 'Adresse' for bpartner created on-the-fly
[#673](metasfresh/metasfresh-webui-frontend-legacy#673) Wrong report file endpoint is called for processes which are started from main menu
[#1347](#1347) technical: ModelClassGenerator shall generate BigDecimal.ZERO instead of Env.ZERO
[#880](#880) Improve IProcessPrecondition framework
[#641](metasfresh/metasfresh-webui-frontend-legacy#641) Shortcuts behavior sometimes weird
[#644](metasfresh/metasfresh-webui-frontend-legacy#644) Creating a new Discount record error
[#649](metasfresh/metasfresh-webui-frontend-legacy#649) windowId shall be handled as a string and not as a number
[#596](metasfresh/metasfresh-webui-frontend-legacy#596) red line appears on document only after refresh
[#433](metasfresh/metasfresh-webui-frontend-legacy#433) ShipmentSchedule: Only displayed after "No data" showing up
[#270](metasfresh/metasfresh-webui-api-legacy#270) cannot add trading unit with batch entry after using add new
[#537](metasfresh/metasfresh-webui-frontend-legacy#537) KPI: implement a non intrusive way to display KPI related errors
[#268](metasfresh/metasfresh-webui-api-legacy#268) HU attributes shall always provide the WidgetType along with the data/changes
[#579](metasfresh/metasfresh-webui-frontend-legacy#579) DocumentNo field is not updated

me-45
hairy tinkerbell struck again
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

5 participants