search

planetfederal / geodroid-server

Apache License 2.0
3 stars 4 forks source link

apps menu causes crash #31

Closed ischneider closed 10 years ago

ischneider commented 10 years ago

Since the service is responsible for creating the directories, it's possible that the 'Apps' menu from the main page will blow up if the service hasn't started (or the directories don't exist)

java.lang.NullPointerException at org.geodroid.server.AppsPage$LoadApps.doInBackground(AppsPage.java:58) at org.geodroid.server.AppsPage$LoadApps.doInBackground(AppsPage.java:36) at android.os.AsyncTask$2.call(AsyncTask.java:287) at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:305) at java.util.concurrent.FutureTask.run(FutureTask.java:137) at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:230) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1076) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:569) at java.lang.Thread.run(Thread.java:864)

ischneider commented 10 years ago

@jdeolive mind checking out abf4ee1 ?

jdeolive commented 10 years ago

Looks good to me. Although I don't see the FilesHelper class anywhere?

On Fri, Nov 22, 2013 at 2:49 PM, Ian Schneider notifications@github.comwrote:

@jdeolive https://github.com/jdeolive mind checking out abf4ee1https://github.com/boundlessgeo/geodroid-server/commit/abf4ee1?

— Reply to this email directly or view it on GitHubhttps://github.com/boundlessgeo/geodroid-server/issues/31#issuecomment-29111923 .

Justin Deoliveira Vice President, Engineering | Boundless jdeolive@boundlessgeo.com @j_deolive https://twitter.com/j_deolive

jdeolive commented 10 years ago

Oops, never mind. see it :)

On Sat, Nov 23, 2013 at 11:15 AM, Justin Deoliveira < jdeolive@boundlessgeo.com> wrote:

Looks good to me. Although I don't see the FilesHelper class anywhere?

On Fri, Nov 22, 2013 at 2:49 PM, Ian Schneider notifications@github.comwrote:

@jdeolive https://github.com/jdeolive mind checking out abf4ee1https://github.com/boundlessgeo/geodroid-server/commit/abf4ee1?

— Reply to this email directly or view it on GitHubhttps://github.com/boundlessgeo/geodroid-server/issues/31#issuecomment-29111923 .

Justin Deoliveira Vice President, Engineering | Boundless jdeolive@boundlessgeo.com @j_deolive https://twitter.com/j_deolive

Justin Deoliveira Vice President, Engineering | Boundless jdeolive@boundlessgeo.com @j_deolive https://twitter.com/j_deolive

ischneider commented 10 years ago

@jdeolive you were right, the initial commit didn't have it :)