Skip to content

Conversation

@lovepoem
Copy link
Member

@lovepoem lovepoem commented Oct 25, 2017

修改 #714 governance/screen/providers/providers页面,对于服务“重复”的错误提示

因为是页面修改,只能手动功能测试

一、问题:多个应用的服务列表页面警告异常(100%重现)

   DUBBO_ADMIN_HOST/governance/screen/providers/providers 

错误页面:
image
(图片一)

二、查看代码逻辑有问题,

#if($providerApplication != $provider.application)  
            #if ($errorLevel != "error") #set($errorLevel = "warn") #end  
            #set($errorMessage = "$errorMessage<br>$i18n.get('warn'): $i18n.get('CheckProviderApplicationDifferent', $providerApplication, $provider.application)")  
#end 
    当判断providerApplication的值和provider.application的值不相同时就给出警告。就成为只要应用名和第一个遍历出来的不一样就加警告。
   需求是:不同应用发了相同服务页面警报。
   修改后就好了。

三、代码修改后前后问题的验证和回归:

1、列表中,不同应用没有提供相同服务,页面修改前后:

错误页面:
见(图片一)

修改后页面:
image
问题已经修复

2、列表中多个应用,有提供相同服务的(页面应该报警)。

image
回归发现功能仍然正确

@chickenlj 合并一下代码

@chickenlj chickenlj merged commit a3c8c62 into apache:master Dec 6, 2017
@lovepoem lovepoem deleted the fix_714_dubbo_admin_console_notice_error_display branch December 7, 2017 05:16
rolandhe pushed a commit to rolandhe/dubbo that referenced this pull request Sep 9, 2019
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.

2 participants