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

属性占位符替换时增加判断 #70

Open
wants to merge 95 commits into
base: property-placeholder-configurer
Choose a base branch
from

Conversation

zzziCode
Copy link

当前属为String类型时,会尝试使用resolvePlaceholder来进行占位符替换,对于直接使用value来指定属性值的属性不会有任何作用:

image-20240311143928710

但是这里没有添加判断,导致重复添加PropertyValue,虽然不会影响后面bean的创建,但是属性列表有重复项也是一种冗余,增加了一定的判断之后就不会出现冗余:

image-20240311151004595

具体请看:#69

DerekYRC and others added 27 commits November 23, 2022 09:35
懒加载和多切面增强
修复了两个小标题没变蓝的错误。
将BeanDefinition 的Map 修改成为并发Map ,并设置初始化大小
初始化方法增加判断是否与InitializingBean方法同名
在README.md的使用方法章节添加了视频教程链接
@DerekYRC
Copy link
Owner

@zzziCode hello,麻烦提交到属性占位符对应的分支吧,我再合并到main分支

@zzziCode zzziCode changed the base branch from main to property-placeholder-configurer March 19, 2024 03:00
@DerekYRC
Copy link
Owner

@zzziCode 在property-placeholder-configurer分支上修复吧,不然更改的文件太多了

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.

9 participants