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

🐛 buildHead 应在 readListener 的 invokeHead 全部执行完成后执行 #2795

Open
wants to merge 2 commits into
base: developing
Choose a base branch
from

Conversation

Hccake
Copy link
Contributor

@Hccake Hccake commented Nov 4, 2022

在多行表头的情况下,readListener#invokeHead 应该被执行多次。

目前的逻辑下,在最后一次执行 invokeHead 前就会执行 buildHead 操作,会导致 invokeHead 失效,需要调整下代码位置

@CLAassistant
Copy link

CLAassistant commented Nov 4, 2022

CLA assistant check
All committers have signed the CLA.

Hccake added a commit to ballcat-projects/ballcat that referenced this pull request Nov 4, 2022
Hccake added a commit to ballcat-projects/ballcat that referenced this pull request Nov 4, 2022
@habyss
Copy link

habyss commented Apr 27, 2023

这个有可能被应用吗? 目前有这种需求 根据一些规则来将表头转换为实体类对应注解的值,以方便导入

@Hccake
Copy link
Contributor Author

Hccake commented Apr 27, 2023

这个有可能被应用吗? 目前有这种需求 根据一些规则来将表头转换为实体类对应注解的值,以方便导入

PR没合并前,可以利用同全类名(包名和类文件名全部一致)的加载顺序优先级来来覆盖原 class,修改后的文件:DefaultAnalysisEventProcessor.java

@habyss
Copy link

habyss commented Apr 28, 2023

这个有可能被应用吗? 目前有这种需求 根据一些规则来将表头转换为实体类对应注解的值,以方便导入

PR没合并前,可以利用同全类名(包名和类文件名全部一致)的加载顺序优先级来来覆盖原 class,修改后的文件:DefaultAnalysisEventProcessor.java

谢谢~

@YangJingf
Copy link

这个有可能被应用吗? 目前有这种需求 根据一些规则来将表头转换为实体类对应注解的值,以方便导入

PR没合并前,可以利用同全类名(包名和类文件名全部一致)的加载顺序优先级来来覆盖原 class,修改后的文件:DefaultAnalysisEventProcessor.java

请问什么时候会被应用呢,我最近去继承AnalysisEventListener,然后修改列名的时候,还是需要利用全类名的方式

@Hccake
Copy link
Contributor Author

Hccake commented Jul 8, 2024

请问什么时候会被应用呢,我最近去继承AnalysisEventListener,然后修改列名的时候,还是需要利用全类名的方式

开发者已经很久没有处理过 PR 了,你可以加他们的钉钉群反馈下。
新的类文件地址: DefaultAnalysisEventProcessor

@YangJingf
Copy link

请问什么时候会被应用呢,我最近去继承AnalysisEventListener,然后修改列名的时候,还是需要利用全类名的方式

开发者已经很久没有处理过 PR 了,你可以加他们的钉钉群反馈下。 新的类文件地址: DefaultAnalysisEventProcessor

您好,请问钉钉群号多少呢

@Hccake
Copy link
Contributor Author

Hccake commented Jul 15, 2024

请问什么时候会被应用呢,我最近去继承AnalysisEventListener,然后修改列名的时候,还是需要利用全类名的方式

开发者已经很久没有处理过 PR 了,你可以加他们的钉钉群反馈下。 新的类文件地址: DefaultAnalysisEventProcessor

您好,请问钉钉群号多少呢

image

easyexcel 的 readme 里面有

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

Successfully merging this pull request may close these issues.

None yet

4 participants