Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

product/packing lookup does nothing when clicking on the single result #744

Closed
teosarca opened this issue May 13, 2017 · 9 comments
Closed

Comments

@teosarca
Copy link
Member

Type of issue

Bug

Steps to reproduce

NOTE: Happens for a product which has only one packing material

  • create new sales order
  • batch entry: try "Convenience salad..." which has only one packing material, the IFCO6410.
  • OK, now click on the field again
  • the dropdown is opened with my "Convenience salad..." product.
  • click on that single dropdown option
  1. nothing happens
  2. while checking the Network tab, remark that each time i click on that the frontend is calling typeahead?query=Convenience ....

Expected behavior

When clicking on that the dropdown shall close, nothing changed.
That's because after selecting the product usually you are asked to pick the packing materials too (the second lookup), but in this case there was only one option so we are not asking.

Of course, in case the second lookup has more then one option, the second lookup's dropdown shall popup next.

image

@teosarca teosarca added this to the 2017-20 milestone May 13, 2017
@damianprzygodzki
Copy link
Contributor

Lookups are refactored. We should postpone it and test after new widget will come.

@metas-mk
Copy link
Member

@damianprzygodzki why postpone? is a functional blocker/ bug.

@damianprzygodzki
Copy link
Contributor

@metas-mk Because @Dunkat is at the end of refactoring lookups. We are going to integrate the brand new widgets soon (~1d) and i believe that it will be faster integrated, than fixed and tested with older widgets.

@teosarca
Copy link
Member Author

Ok, so it shall be fixed by #600

@cadavre
Copy link
Contributor

cadavre commented May 23, 2017

@metas-lc tested on new lookups?

@teosarca
Copy link
Member Author

To be tested after #600

@metas-mk
Copy link
Member

Tested. The following issues occurred:
a) First product row, Product chosen, packing item is shown as a drop-down. This product only has 1 Packing Item, the drop-down is shown. Because it's the only line, the user just has to press enter and tab and can record the qty. OK
b) Create the second row with the same product as the first row, Product is chosen, [enter], then the packing dropdown does not appear. NOK (should be same as a) )
c) Create the next row with a product which does not have any Packing material. The Product is chosen. The dropdown for Packing Material is opened but empty. The user cannot get out of that dropdown w/ keyboard. NOK (should be like in case a) press [enter] w/ empty selection.

image

metas-ts added a commit to metasfresh/metasfresh that referenced this issue Jun 1, 2017
[#819](metasfresh/metasfresh-webui-frontend-legacy#819) HTTP header "Accept-Language" shall be correctly set
[#1663](#1663) new order checkup jasper
[#1687](#1687) Refactor Request Tab in Partner Window to allow zoom and advanced edit
[#1239](#1239) Barcode on Bestellkontrolle report
[#428](metasfresh/metasfresh-webui-api-legacy#428) internal: Include AD_Language in layout ETags
[#376](metasfresh/metasfresh-webui-api-legacy#376) Hide Roles on Logon that are not ideal for WebUI
[#426](metasfresh/metasfresh-webui-api-legacy#426) internal: JSONDocument - drop "fields" array field
[#600](metasfresh/metasfresh-webui-frontend-legacy#600) Lookup revamp – collective issue
[#425](metasfresh/metasfresh-webui-api-legacy#425) API support for bookmarking menu items
[#86](metasfresh/metasfresh-webui-api-legacy#86) Check and get rid of "Parameter 'UOMConversion' not found in context" console warnings
[#812](metasfresh/metasfresh-webui-frontend-legacy#812) 1 Key Press save action error during typing
[#1701](#1701) Customer Return Window additional Fields
[#801](metasfresh/metasfresh-webui-frontend-legacy#801) Included row: show row's references in context menu
[#744](metasfresh/metasfresh-webui-frontend-legacy#744) product/packing lookup does nothing when clicking on the single result
[#1640](#1640) fix prepare_services_superuser.sh
[#810](metasfresh/metasfresh-webui-frontend-legacy#810) quickInput endpoints are wrongly called
[#415](metasfresh/metasfresh-webui-api-legacy#415) Notification for Vendor returns jumps to wrong window
[#418](metasfresh/metasfresh-webui-api-legacy#418) Disable zoom into string lookups until it's fixed
[#1644](#1644) Add email validator syntax
[#410](metasfresh/metasfresh-webui-api-legacy#410) Provide view row field zoom-into endpoint

me-45
damianprzygodzki added a commit that referenced this issue Jun 1, 2017
product/packing lookup does nothing when clicking on the single result #744
@Dunkat
Copy link
Contributor

Dunkat commented Jun 1, 2017

Fixed:
a,b) when only one item is available, should be selected and no dropdown shown, tested on 'Convenience salad'
c) when product does not have any Packing material press enter to close dropdown

@metas-rc metas-rc self-assigned this Jun 9, 2017
@metas-rc
Copy link
Member

metas-rc commented Jun 9, 2017

IT
Tested on webui

  • selecting product with only 1 packing material defined
  • => OK: The dropdown is no longer displayed
  • => OK: Press tab and get directly to the qty
  • Selecting product with no packing material
  • => OK: I get the message bellow the product that " There is no choice available"
  • => OK: I press enter, then tab => I get to the qty field
  • For product with more than 1 packing material, the default one is created but I can only choose another one using the mous. A followup task is already in progress and will be integrated for the next release.
  • For products with packing materials defined, I have no posibility to choose "No packing item" if I use the batch entry. I can do it, nevertheless, if I use the Add new button.
    Talked with Mark and this will also be covered in a follow-up task about harmonizing Add New with Batch entry

@metas-rc metas-rc closed this as completed Jun 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants