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

CMV Editor widget issue in WAB enabled cmv viewer since api 3.21 #30

Closed
ERS-Long opened this issue Sep 25, 2018 · 7 comments
Closed

CMV Editor widget issue in WAB enabled cmv viewer since api 3.21 #30

ERS-Long opened this issue Sep 25, 2018 · 7 comments

Comments

@ERS-Long
Copy link

Today a user reported to us this issue, if we update the viewer to 3.25, then the CMV editor widget stops working, the error popup is

Uncaught RangeError: Maximum call stack size exceeded

If we switch back to 3.21, it still works, tried 3.22, 3.23, 3.24, 3.25, and 3.26, all failed. Please see if you experienced the same or not, any suggestion on how to fix this issue.

Thanks!

@tmcgee
Copy link
Member

tmcgee commented Sep 26, 2018

@ERS-Long Which version of CMV and which version of Web App Builder?

@ERS-Long
Copy link
Author

I tried most current CMV (3.26) with WAB 2.9. (if change api from 3.26 to 3.21, then no error).

Thanks,

@tmcgee
Copy link
Member

tmcgee commented Sep 27, 2018

@ERS-Long Are you using the CMV Identify widget with:

defaultMapClickMode: 'identify',

in the config. file?

Changes in the Esri Editor Toolbar are causing the endless recursion. It is in confict with this section of the WAB Mixin. Presently, I don't have a work-around for all cases. But if you are not using identify, you can possibly solve this by removing the defaultMapClickMode or commenting out the section of code in the MixIn. I can verify this solves the issue but may not be acceptable for your application.

@ERS-Long
Copy link
Author

@tmcgee Got it, that is why. Yes, I can comment out the wab mixin map click section code.

Thanks for your tip, as aways, it just works!!

:)

@tmcgee
Copy link
Member

tmcgee commented Sep 27, 2018

@ERS-Long When using this cmv-wab-widgets, another consideration is to use the WAB Edit widget. It doesn't work in the current version of cmv-wab-widgets for reasons not directly related to the Edit widget itself. Thanks to your question, I have explored the issue and have the Edit widget working locally. 😄

@ERS-Long
Copy link
Author

ERS-Long commented Sep 27, 2018

Good to know, try to stay away from default WAB widgets as far as possible, use them only when i have to as the last resort. :) That being said, I tried to load the WAB default edit widget in, no luck, when you have time if you could share your trick to get that work in CMV, that will be great. But not urgent, only when you have time. Thanks!

@tmcgee
Copy link
Member

tmcgee commented Sep 27, 2018

@ERS-Long The Smart Editor widget is another more "evolved" WAB widget for editing. It is a newer widget so is more robust than the WAB and CMV Editor widgets that rely on the antiquated Esri Editor Toolbar.

I have that widget working as well.

Some time soon, I will commit the changes into this repo to allow these widgets to work.

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

No branches or pull requests

2 participants