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

Sharing via other apps cause crash #1622

Closed
gerroon opened this issue Apr 21, 2016 · 5 comments
Closed

Sharing via other apps cause crash #1622

gerroon opened this issue Apr 21, 2016 · 5 comments

Comments

@gerroon
Copy link

gerroon commented Apr 21, 2016

The latest beta version crashes when trying to share files from other apps. In the previous version sharing did not seem to work. By sharing I mean sending files from other apps to OWncloud app for uploading. I am not sure if this is not a supported feature at all.

Expected behaviour

Tell us what should happen
WHen I share an image or a file to upload from another app, I expect it to upload to the selected folder

Actual behaviour

Tell us what happens instead
Owncloud crashes when a file is sent to OC app via share menu from other apps

Steps to reproduce

  1. select an image from gallery in gallery app
  2. share it with Owncloud

Can this problem be reproduced with the official owncloud server?
(url: https://demo.owncloud.org, user: test, password: test)
No idea

Environment data

Android version:

Device model:

Stock or customized system:

ownCloud app version:
beta-20160419

ownCloud server version:
ownCloud 8.2.2 (stable)

Logs

java.lang.RuntimeException: Unable to start activity ComponentInfo{com.owncloud.android.beta/com.owncloud.android.ui.activity.Uploader}: java.lang.IllegalStateException: This Activity already has an action bar supplied by the window decor. Do not request Window.FEATURE_SUPPORT_ACTION_BAR and set windowActionBar to false in your theme to use a Toolbar instead.
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3253)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3349)
at android.app.ActivityThread.access$1100(ActivityThread.java:221)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1794)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:158)
at android.app.ActivityThread.main(ActivityThread.java:7224)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)
Caused by: java.lang.IllegalStateException: This Activity already has an action bar supplied by the window decor. Do not request Window.FEATURE_SUPPORT_ACTION_BAR and set windowActionBar to false in your theme to use a Toolbar instead.
at android.support.v7.app.AppCompatDelegateImplV7.setSupportActionBar(AppCompatDelegateImplV7.java:198)
at android.support.v7.app.AppCompatActivity.setSupportActionBar(AppCompatActivity.java:99)
at com.owncloud.android.ui.activity.ToolbarActivity.setupToolbar(ToolbarActivity.java:52)
at com.owncloud.android.ui.activity.Uploader.populateDirectoryList(Uploader.java:433)
at com.owncloud.android.ui.activity.Uploader.onAccountSet(Uploader.java:201)
at com.owncloud.android.ui.activity.BaseActivity.onStart(BaseActivity.java:189)
at com.owncloud.android.ui.activity.FileActivity.onStart(FileActivity.java:173)
at android.app.Instrumentation.callActivityOnStart(Instrumentation.java:1265)
at android.app.Activity.performStart(Activity.java:6915)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3216)
... 9 more

************ DEVICE INFORMATION ***********
Brand: samsung
Model: SAMSUNG-SM-G930A

@tobiasKaminsky
Copy link
Contributor

@AndyScherzinger this is ActionBar AppCompat related.
Can you have a look, please? :)

@AndyScherzinger AndyScherzinger self-assigned this Apr 22, 2016
@AndyScherzinger
Copy link
Contributor

AndyScherzinger commented Apr 22, 2016

Yes that is related to beta and my toolbar refactoring. I'll take care of it on the weekend! 👍
Sorry for the inconvenience!
Fix to be done for #1559 / beta

@strk
Copy link

strk commented Apr 24, 2016

I'm just starting with owncloud and also hit this issue, looking forward to your fix @AndyScherzinger, thanks a lot !

AndyScherzinger added a commit that referenced this issue Apr 24, 2016
@AndyScherzinger
Copy link
Contributor

AndyScherzinger commented Apr 24, 2016

@tobiasKaminsky and @strk I pushed a fix to the repo: be52cd7

So this will be fixed with the next beta release :)

I missed this in the beta merge, the bug isn't present on the feature branch itself.

@AndyScherzinger AndyScherzinger removed their assignment Apr 24, 2016
@AndyScherzinger
Copy link
Contributor

will be in next beta

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

No branches or pull requests

4 participants