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

Console error when grid view updates #1542

Closed
metas-lc opened this issue Jan 30, 2018 · 5 comments
Closed

Console error when grid view updates #1542

metas-lc opened this issue Jan 30, 2018 · 5 comments

Comments

@metas-lc
Copy link

Is this a bug or feature request?

bug

What is the current behavior?

Console error when grid view updates TypeError: Cannot read property 'ID' of undefined

Which are the steps to reproduce?

  1. open settings:
    https://w101.metasfresh.com:8443/window/53100/2188223

  2. new tab: open user window in grid view (this is page no. 2 because there was my user from settings)
    https://w101.metasfresh.com:8443/window/108?page=2&viewId=108-934e04bf3a09430ba73fe8bd159d302e

  3. go back to settings tab and change email address

  4. go back to user tab, see console
    => ViewActions.js:153 Uncaught (in promise) TypeError: Cannot read property 'ID' of undefined
    at Object.values.map.viewRowField (ViewActions.js:153)
    at Array.map ()
    at mergeColumnInfosIntoViewRow (ViewActions.js:153)
    at toRows.map.row (ViewActions.js:196)
    at Array.map ()
    at mergeRows (ViewActions.js:196)
    at then.response (DocumentList.js:229)
    at
    => also you need to refresh to see the change NOK

What is the expected or desired behavior?

no errors, and the change shall appear without refreshing

@siemiatj
Copy link
Contributor

@metas-lc Tested with latest master. I don't get the error.
@teosarca When I changed my email address in one tab, in the users list tab I got this:

screen shot 2018-03-29 at 8 04 28 pm

No idea if the list should get updated, or what are the requirements here.

@teosarca
Copy link
Member

did a quick check, it seems a backend bug. On it.

@teosarca
Copy link
Member

teosarca commented Mar 29, 2018

fixed by metasfresh/metasfresh#3793

@teosarca
Copy link
Member

teosarca commented Mar 29, 2018

reopening because metasfresh/metasfresh#3793 is fixing the cache invalidation issue and not the Cannot read property 'ID' of undefined issue.
That one shall be checked again by IT guys.

@teosarca teosarca reopened this Mar 29, 2018
@metas-lc
Copy link
Author

IT
repeated the testcase above => works fine, no errors and the field is updated without using refresh OK

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

3 participants