Named BeanDefinition not using 'primary_type'

Describe the bug When defining two objects with same named string, it fails with DefinitionOverrideException

To Reproduce Steps to reproduce the behavior:

1. Create definitions like:

factory(named("statusOne")){ StatusObject(get(named("statusOne"))) }
single(named("statusOne")){ StatusObjectConstructorParam() }

1. start app… startKoin{...}

Expected behavior named definitions also check primary_type to determine object definition

Koin project used and used version (please complete the following information):

    implementation "org.koin:koin-androidx-scope:2.0.1"
    implementation "org.koin:koin-androidx-viewmodel:2.0.1"
    implementation "org.koin:koin-androidx-ext:2.0.1"

this appeard after migrating from 1.0.2 to 2.0.1 - on 1.x it such definitions were working fine, on 2.x every named (eg. based on enums) definitions should be changed, which is painful job 😉 or there is some other way I didn’t in docs?

[arnaudgiuliani]

Hello @bovquier,

this behavior is intended: a qualifier is unique for all definitions. What is your use case behind reusing the same qualifier?

[ghost]

@arnaudgiuliani the reason is eg. I’m using ItemKeyedDataSource and DataSource.Factory from jetpack paging, and same implementations for few source types (eg. based on item status), this doesn’t allow me to make them defined in simple way like in Kotlin1 (have such setup for that). In Koin2, this will need to create ActiveDataSource/PendingDataSource/LockedDataSource, to match the requirements. In Koin1 I have something like:

        MyProjectsAdapter.ProjectsPagesItems.values().forEach { page -> single(page.name) { ProjectsDataSourceFactory() } }
        MyProjectsAdapter.ProjectsPagesItems.values().forEach { page -> factory(page.name) { ProjectsDataSourceManager(get(page.name)) } }

and I have 2 lines, and do not think about how many PagesTypes there will be, all adapter pages will have its sources injected. But couldn’t achive that in K2, but I may be missing something in docs, and you, guys, can give some hint, if there is such simple approach for that.