fix: nacos property arg bug

[shouyuwang]

When I have configurations with the same name in different yaml files,the result obtained through @NacosConfigurationProperties may be a bug.

[shouyuwang]

FindBug Test

Test1Config

@Configuration
@NacosConfigurationProperties(groupId = "test1", dataId = "test1.yaml", autoRefreshed = true)
public class Test1Config {

    /**
     *  test
     */
    private String test;


    public String getTest() {
        return test;
    }

    public void setTest(String test) {
        this.test = test;
    }
}

Test2Config

@Configuration
@NacosConfigurationProperties(groupId = "test2", dataId = "test2.yaml", autoRefreshed = true)
public class Test2Config {

    /**
     * test
     */
    private String test;


    public String getTest() {
        return test;
    }

    public void setTest(String test) {
        this.test = test;
    }
}

TestsClass

@EnableScheduling
@SpringBootApplication
public class NacosTestApplication {

    public static void main(String[] args) {
        SpringApplication.run(NacosTestApplication.class, args);
    }

    private Logger logger = LoggerFactory.getLogger(NacosTestApplication.class);

    @Autowired
    private Test1Config test1Config;

    @Autowired
    private Test2Config test2Config;

    @Scheduled(cron = "0/5 * * * * ?")
    public void print(){
        logger.info("test1:{}, test2:{}", test1Config.getTest(), test2Config.getTest());
    }
}

Nacos test1.yaml

Nacos test2.yaml

Test RunResult

test1.yaml update

test1 change result

test2.yaml update

test2 change result

FixBug Test

NacosPropertySourceConstruction Mthod is

public NacosPropertySource(String dataId, String groupId, String name,
			String nacosConfig, String type) {
  super(name, toProperties(dataId, groupId, nacosConfig, type));
  this.type = type;
}

But in NacosBootConfigurationPropertiesBinder doBindmethod useNacosPropertySource propertySource = new NacosPropertySource(name, dataId, groupId, content, configType);Maybe the name is not a right order.environment.getPropertySources().remove(name) environment can not remove the name. So I change to NacosPropertySource propertySource = new NacosPropertySource(dataId, groupId, name, content, configType);

Test Result

test2.yaml update

test2 change result

test1.yaml update

test1 change result

[shouyuwang]

@xuechaos please take a look, if you have time,thanks.