bumptech/glide

Annotation processor uses class name as package name causing conflicts

Open

#2 860 ouverte le 1 févr. 2018

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)Java (6 176 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (35 023 stars)
Métriques de merge PR
 (Merge moyen 4j 10h) (11 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examiner comment AppModuleProcessor génère le nom du package pour la classe GlideApp. Vérifiez si le nom du package est dérivé du nom de la classe englobante et assurez vous qu'il n'entre pas en conflit avec des packages existants.
Stack technique
javaandroid
Domaine
mobile
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaAndroidannotation processing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70