Skip to content

Conversation

@TranceLove
Copy link
Collaborator

OK, I too think it should have been fixed somewhere else, but now I got one... whatever.

Description

Icon should be androidx.appcompat.widget.AppCompatImageView instead.

Issue tracker

Manual tests

  • Done

  • Device: Pixel 7 emulator

  • OS: Android 14
    SMB search dialog should not crash with ClassCastException error when opening.

Build tasks success

Successfully running following tasks on local:

  • ./gradlew assembledebug
  • ./gradlew spotlessCheck

Icon should be androidx.appcompat.widget.AppCompatImageView instead
@TranceLove TranceLove self-assigned this Oct 15, 2023
@TranceLove TranceLove added Issue-Bug Related unexpected behavior or something worth investigating. Area-SMB Related to SMB connections. labels Oct 15, 2023
@TranceLove TranceLove added this to the v4.0 milestone Oct 15, 2023
Copy link
Member

@VishnuSanal VishnuSanal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#3885 is a StringIndexOutOfBoundsException, not ClassCastException 🤔

@TranceLove
Copy link
Collaborator Author

#3885 is a StringIndexOutOfBoundsException, not ClassCastException 🤔

Awch. I thought I tagged the other PR there... this PR is not related to any reported issues - just found a crash as I walked through #3885 though.

@VishnuSanal VishnuSanal added the not-yet-rolled An issue detected on a feature after it is implemented but before it is rolled. label Oct 15, 2023
@VishalNehra VishalNehra merged commit 0a20267 into release/4.0 Oct 30, 2023
@VishalNehra VishalNehra deleted the bugfix/smb-search-dialog-wrong-class branch October 30, 2023 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-SMB Related to SMB connections. Issue-Bug Related unexpected behavior or something worth investigating. not-yet-rolled An issue detected on a feature after it is implemented but before it is rolled.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants