eng-Robert / acra-reporter

Automatically exported from code.google.com/p/acra-reporter
0 stars 0 forks source link

New App dialog not working #4

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Deployed acra-reporter Appengine project as a Single user
2. Clicked on "New app" button
3. Entered app information in the Package and App name fields
4. Pressed Ok

What is the expected output? What do you see instead?
The dialog would be dismissed and it would start recieving acra crash reports 
from the specified app.

What version of the product are you using? On what operating system?
Built from the latest available version of the source and using the latest 
build of ACRA.

I also setup my android app to send a report to the appengine project and it 
appears to have successfully sent the report (according to acra logcat 
messages).

Original issue reported on code.google.com by gab...@gmail.com on 27 Feb 2013 at 8:52

GoogleCodeExporter commented 9 years ago
I have the same problem. Here's a javascript stack trace of the presumed error.

Uncaught com.google.gwt.event.shared.UmbrellaException: Exception caught: 
(TypeError) 
 stack: TypeError: Cannot read property 'b' of null
    at vWb (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2568:203)
    at Object.VWb [as Xb] (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2837:19099)
    at Object.cs (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2828:25093)
    at zu (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2685:190)
    at ou (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2467:61)
    at OX (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:968:26)
    at Ir (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2311:102)
    at QX (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2229:105)
    at Object._X (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2831:8241)
    at _4 (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:1980:73)
    at HTMLButtonElement.<anonymous> (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2812:223)
    at Pj (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:1319:29)
    at Sj (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:1782:44)
    at HTMLButtonElement.<anonymous> (http://xxx.appspot.com/acrareporter/8E182221511124E93BB2D17B7884CF9F.cache.html:2022:45): Cannot read property 'b' of null 

Original comment by j...@bekas.org on 7 Mar 2013 at 10:22

GoogleCodeExporter commented 9 years ago
Same error occurs on https://wintersacrareporter.appspot.com/ so I can't create 
an App there either. :(

Original comment by j...@bekas.org on 7 Mar 2013 at 11:49

GoogleCodeExporter commented 9 years ago
Hey Pals any solution you found for this issue.

Original comment by tree.h...@gmail.com on 28 Mar 2013 at 3:52

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
I've done a bit of investigation on this and the issue is that the "Map files 
to keep:" (aka textMapsToKeep) field is not being bound correctly and so when 
we come to read from it, it's null and we get a NullPointerException.  The full 
(unobfuscated) stack is below.

11:20:32.711 [ERROR] [acrareporter] Uncaught exception escaped

com.google.gwt.event.shared.UmbrellaException: Exception caught: null
    at com.google.gwt.event.shared.HandlerManager.fireEvent(HandlerManager.java:129)
    at com.google.gwt.user.client.ui.Widget.fireEvent(Widget.java:129)
    at com.google.gwt.event.dom.client.DomEvent.fireNativeEvent(DomEvent.java:116)
    at com.google.gwt.user.client.ui.Widget.onBrowserEvent(Widget.java:177)
    at com.google.gwt.user.client.DOM.dispatchEventImpl(DOM.java:1351)
    at com.google.gwt.user.client.DOM.dispatchEvent(DOM.java:1307)
    at sun.reflect.GeneratedMethodAccessor41.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:601)
    at com.google.gwt.dev.shell.MethodAdaptor.invoke(MethodAdaptor.java:103)
    at com.google.gwt.dev.shell.MethodDispatch.invoke(MethodDispatch.java:71)
    at com.google.gwt.dev.shell.OophmSessionHandler.invoke(OophmSessionHandler.java:172)
    at com.google.gwt.dev.shell.BrowserChannelServer.reactToMessagesWhileWaitingForReturn(BrowserChannelServer.java:338)
    at com.google.gwt.dev.shell.BrowserChannelServer.invokeJavascript(BrowserChannelServer.java:219)
    at com.google.gwt.dev.shell.ModuleSpaceOOPHM.doInvoke(ModuleSpaceOOPHM.java:136)
    at com.google.gwt.dev.shell.ModuleSpace.invokeNative(ModuleSpace.java:571)
    at com.google.gwt.dev.shell.ModuleSpace.invokeNativeObject(ModuleSpace.java:279)
    at com.google.gwt.dev.shell.JavaScriptHost.invokeNativeObject(JavaScriptHost.java:91)
    at com.google.gwt.core.client.impl.Impl.apply(Impl.java)
    at com.google.gwt.core.client.impl.Impl.entry0(Impl.java:242)
    at sun.reflect.GeneratedMethodAccessor37.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:601)
    at com.google.gwt.dev.shell.MethodAdaptor.invoke(MethodAdaptor.java:103)
    at com.google.gwt.dev.shell.MethodDispatch.invoke(MethodDispatch.java:71)
    at com.google.gwt.dev.shell.OophmSessionHandler.invoke(OophmSessionHandler.java:172)
    at com.google.gwt.dev.shell.BrowserChannelServer.reactToMessages(BrowserChannelServer.java:293)
    at com.google.gwt.dev.shell.BrowserChannelServer.processConnection(BrowserChannelServer.java:547)
    at com.google.gwt.dev.shell.BrowserChannelServer.run(BrowserChannelServer.java:364)
    at java.lang.Thread.run(Thread.java:722)
Caused by: java.lang.NullPointerException: null
    at nz.org.winters.appspot.acrareporter.client.ui.PackageEdit.onButtonOKClick(PackageEdit.java:116)
    at nz.org.winters.appspot.acrareporter.client.ui.PackageEdit_PackageEmailTemplateUiBinderImpl$Widgets$1.onClick(PackageEdit_PackageEmailTemplateUiBinderImpl.java:51)
    at com.google.gwt.event.dom.client.ClickEvent.dispatch(ClickEvent.java:56)
    at com.google.gwt.event.dom.client.ClickEvent.dispatch(ClickEvent.java:1)
    at com.google.gwt.event.shared.GwtEvent.dispatch(GwtEvent.java:1)
    at com.google.web.bindery.event.shared.EventBus.dispatchEvent(EventBus.java:40)
    at com.google.web.bindery.event.shared.SimpleEventBus.doFire(SimpleEventBus.java:193)
    at com.google.web.bindery.event.shared.SimpleEventBus.fireEvent(SimpleEventBus.java:88)
    at com.google.gwt.event.shared.HandlerManager.fireEvent(HandlerManager.java:127)
    at com.google.gwt.user.client.ui.Widget.fireEvent(Widget.java:129)
    at com.google.gwt.event.dom.client.DomEvent.fireNativeEvent(DomEvent.java:116)
    at com.google.gwt.user.client.ui.Widget.onBrowserEvent(Widget.java:177)
    at com.google.gwt.user.client.DOM.dispatchEventImpl(DOM.java:1351)
    at com.google.gwt.user.client.DOM.dispatchEvent(DOM.java:1307)
    at sun.reflect.GeneratedMethodAccessor41.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:601)
    at com.google.gwt.dev.shell.MethodAdaptor.invoke(MethodAdaptor.java:103)
    at com.google.gwt.dev.shell.MethodDispatch.invoke(MethodDispatch.java:71)
    at com.google.gwt.dev.shell.OophmSessionHandler.invoke(OophmSessionHandler.java:172)
    at com.google.gwt.dev.shell.BrowserChannelServer.reactToMessagesWhileWaitingForReturn(BrowserChannelServer.java:338)
    at com.google.gwt.dev.shell.BrowserChannelServer.invokeJavascript(BrowserChannelServer.java:219)
    at com.google.gwt.dev.shell.ModuleSpaceOOPHM.doInvoke(ModuleSpaceOOPHM.java:136)
    at com.google.gwt.dev.shell.ModuleSpace.invokeNative(ModuleSpace.java:571)
    at com.google.gwt.dev.shell.ModuleSpace.invokeNativeObject(ModuleSpace.java:279)
    at com.google.gwt.dev.shell.JavaScriptHost.invokeNativeObject(JavaScriptHost.java:91)
    at com.google.gwt.core.client.impl.Impl.apply(Impl.java)
    at com.google.gwt.core.client.impl.Impl.entry0(Impl.java:242)
    at sun.reflect.GeneratedMethodAccessor37.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:601)
    at com.google.gwt.dev.shell.MethodAdaptor.invoke(MethodAdaptor.java:103)
    at com.google.gwt.dev.shell.MethodDispatch.invoke(MethodDispatch.java:71)
    at com.google.gwt.dev.shell.OophmSessionHandler.invoke(OophmSessionHandler.java:172)
    at com.google.gwt.dev.shell.BrowserChannelServer.reactToMessages(BrowserChannelServer.java:293)
    at com.google.gwt.dev.shell.BrowserChannelServer.processConnection(BrowserChannelServer.java:547)
    at com.google.gwt.dev.shell.BrowserChannelServer.run(BrowserChannelServer.java:364)
    at java.lang.Thread.run(Thread.java:722)

The textMapsToKeep field seems to be defined correctly in PackageEdit.ui.xml 
but, unusually, it is an IntegerBox - most other fields are TextBoxes.  The GWT 
docs for IntegerBox say: "Experimental API: This class is still under rapid 
development, and is very likely to be deleted. Use it at your own risk.", so 
this could have changed recently - I'm compiling with GWT 2.5.1 and the docs 
say to use 2.5.0 (although I can't find that now).

Anyway, the workaround seems to be to comment out lines 99 and 116 of 
src/nz/org/winters/appspot/acrareporter/client/ui/PackageEdit.java:

    // textMapsToKeep.setValue(aps.mappingsToKeep);

    // appPackage.mappingsToKeep = textMapsToKeep.getValue();

I can now create Apps and view their state.  Note that this workaround doesn't 
allow you to change the "Map files to keep" property, but this defaults to 10, 
which is a reasonably sensible value. 

Original comment by maffy...@gmail.com on 30 Mar 2013 at 11:39

GoogleCodeExporter commented 9 years ago
Thankyou very much for the quick workaround, now i can add the new app. Thanks 
a million

Original comment by tree.h...@gmail.com on 30 Mar 2013 at 12:01

GoogleCodeExporter commented 9 years ago
Hello one more question, After i made this change might be something goes 
worng: From where i can see the stack trace reports as now when i am clicking 
on Open for the app name  its open the crash report but no Data is loading on 
that page. is this is default behavior or i am missing any step.

Original comment by tree.h...@gmail.com on 30 Mar 2013 at 12:11

GoogleCodeExporter commented 9 years ago
I'm not an expert on this code.  (The first time I looked at it was this 
morning when I hit this issue.)

However, for me,
- after clicking Open next to my app's name, I'm taken to a new view with an 
Errors heading on the left and a Report heading on the right
- at this stage, no data is loaded - I must click on a specific entry below the 
Errors heading to bring up a crash report
- if I then click on subheadings (e.g. Device, Settings), I see data
- the Stack subheading is a bit odd - by default, the "Remapped stack trace" 
section is open, and since my stack traces don't need remapping, this section 
is blank - if I close the "Remapped stack trace" section, I see an "Original 
stack Strace" section, which contains my stack.

Does this help/make sense?

Original comment by maffy...@gmail.com on 30 Mar 2013 at 3:40

GoogleCodeExporter commented 9 years ago
So, I have just figured out that I don't get an e-mail when new issues are 
created - apologies for not seeing this earlier..

I have not seen this problem, on the acrareporter appengine instance - I am 
puzzled. I used the integer input to avoid non-integer input. I have used it 
elsewhere without issues.

I do need to make a change to collapse the remapped stack trace when there is 
none (All my apps are proguarded), I will get onto it next week, I will have 
some spare time then..

Original comment by mat...@winters.org.nz on 4 Apr 2013 at 9:49

GoogleCodeExporter commented 9 years ago
No problem on not noticing this issue - thanks for all your development efforts 
- it's a really useful package!

I'm suspicious that the IntegerBox issue could be due to a change between GWT 
2.5.0 and GWT 2.5.1.  Unfortunately, I can't find a download link for that now 
(Eclipse only offers the latest and greatest).  Alternatively, maybe it's a 
result of doing a completely clean (i.e. straight from svn) build?

(On the ProGuard point, I should really be doing this on my app, but it's still 
very much in alpha - I'm using acrareporter so that I can get decent diags from 
the tests that my (less tech-savvy) colleague is running.)

Thanks again - I've found this really useful!

Original comment by maffy...@gmail.com on 7 Apr 2013 at 1:15

GoogleCodeExporter commented 9 years ago
Just updated the instance and the code..

Fixed the map files to keep - just needed a default value and a check on empty 
entry.
Added a refresh to the app list after an app is added.
Added collapse of the mapped stack trace when there is none.
Some minor fixes.

Original comment by mat...@winters.org.nz on 11 Apr 2013 at 11:52