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

Auto import of dart file does not work with mixins declared with mixin keyword. #3204

Closed
RussellZornes opened this issue Feb 16, 2019 · 2 comments
Assignees
Milestone

Comments

@RussellZornes
Copy link

Steps to Reproduce

When you attempt to auto import a class name that is declared with the mixin keyword, the name of the file does not appear in the popup.

// some_mixin.dart
mixin SomeMixin {}

// foo.dart
class Foo extends Object with SomeMixin <- option return on mixin name

Expected:
The file some_mixin.dart would appear in popup so you can add the import.

Results:
No file appears in popup to auto import.

Notes:
It works if the mixin is declared as an abstract class rather than with the mixin keyword however.

Version info

[√] Flutter (Channel dev, v1.2.0, on Microsoft Windows [Version 10.0.17134.590], locale en-US)
• Flutter version 1.2.0 at C:\dev\libraries\flutter2\flutter
• Framework revision 06b979c4d5 (3 weeks ago), 2019-01-25 14:27:35 -0500
• Engine revision 36acd02c94
• Dart version 2.1.1 (build 2.1.1-dev.3.2 f4afaee422)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at C:\dev\libraries\android-sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• ANDROID_HOME = C:\dev\libraries\android-sdk
• Java binary at: C:\dev\tools\android-studio-ide-171.4402976-windows\android-studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
• All Android licenses accepted.

[√] Android Studio (version 3.3)
• Android Studio at C:\dev\tools\android-studio-ide-171.4402976-windows\android-studio
• Flutter plugin version 33.0.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] IntelliJ IDEA Ultimate Edition (version 2018.1)
• IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2018.1
• Flutter plugin version 23.1.2
• Dart plugin version 173.4700

@stevemessick
Copy link
Member

@scheglov Does this sound familiar?

@stevemessick stevemessick added this to the On Deck milestone Feb 19, 2019
@scheglov scheglov self-assigned this Feb 20, 2019
@scheglov
Copy link
Contributor

dart-bot pushed a commit to dart-lang/sdk that referenced this issue Feb 20, 2019
…ick Fix.

R=brianwilkerson@google.com

Bug: flutter/flutter-intellij#3204
Change-Id: Ia80fca430c9cbe710ccaf74b97a1b025634b85b1
Reviewed-on: https://dart-review.googlesource.com/c/93641
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
dart-bot pushed a commit to dart-lang/sdk that referenced this issue Feb 20, 2019
However this is not what the issue is about.
But still a bug.

R=brianwilkerson@google.com

Bug: flutter/flutter-intellij#3204
Change-Id: I743e1684f84844422bdfc8fcb30c2b3a051f4580
Reviewed-on: https://dart-review.googlesource.com/c/93376
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
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

3 participants