bumptech/glide

Annotation processor uses class name as package name causing conflicts

Open

#2860 aperta il 1 feb 2018

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Java (6176 fork)batch import
bughelp wanted

Metriche repository

Star
 (35.023 star)
Metriche merge PR
 (Merge medio 4g 10h) (11 PR mergiate in 30 g)

Descrizione

Glide Version: 4.x

Integration libraries: N/A

Device/Android Version: N/A

Issue details / Repro steps / Use case background:

https://github.com/bumptech/glide/blob/eff228fcafc2a3e6077d15d0f2a995edf46f1f56/annotation/compiler/src/main/java/com/bumptech/glide/annotation/compiler/AppModuleProcessor.java#L84

See https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/glidelibrary/ZreirxaqMu0/QGhfJKJoBAAJ

Related: https://github.com/bumptech/glide/issues/2512

Glide load line / GlideModule (if any) / list Adapter code (if any):

package com.example;
class MyApp {
    @GlideModule
    static class MyModule {
    }
}

will generate com.example.MyApp.GlideApp; where MyApp will be a package and this conflicts with the outer class from the app.

Guida contributor

Direzione di ricerca
Investigare come AppModuleProcessor genera il nome del pacchetto per la classe GlideApp. Verificare se il nome del pacchetto deriva dal nome della classe contenitore e assicurarsi che non entri in conflitto con pacchetti esistenti.
Tech stack
javaandroid
Dominio
mobile
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaAndroidannotation processing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70