bumptech/glide

Annotation processor uses class name as package name causing conflicts

Open

#2.860 aberto em 1 de fev. de 2018

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)Java (6.176 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (35.023 stars)
Métricas de merge de PR
 (Mesclagem média 4d 10h) (11 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue como o AppModuleProcessor gera o nome do pacote para a classe GlideApp. Verifique se o nome do pacote é derivado do nome da classe externa e garanta que não entre em conflito com pacotes existentes.
Pilha de tecnologia
javaandroid
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaAndroidannotation processing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70