The typo is there because the same typo exists when creating the search
builder in the constructor. In the constructor, the string
"serviceProvderType" is associated with the VO field 'providerName'. If
you fix the method, you should fix the constructor (although there is no
apparent logic error, just confused naming)
On 6/4/13 6:40 PM, "Soheil Eizadi" <seiz...@infoblox.com> wrote:
>I am looking at the code for findByServiceProvider in
>PhysicalNetworkServiceProviderDaoImpl.java:
>
> public PhysicalNetworkServiceProviderVO findByServiceProvider(long
>physicalNetworkId, String providerType) {
> SearchCriteria<PhysicalNetworkServiceProviderVO> sc =
>physicalNetworkServiceProviderSearch.create();
> sc.setParameters("physicalNetworkId", physicalNetworkId);
> sc.setParameters("serviceProvderType", providerType);
> return findOneBy(sc);
> }
>
>I don't see a field serviceProvderType (even if I fix the typo!) Am I
>missing something, it seems like this code is being used by the
>Networking code in the plugins.
>
>Seems like this should be changed to:
> public PhysicalNetworkServiceProviderVO findByServiceProvider(long
>physicalNetworkId, String providerName) {
>......
> sc.setParameters("providerName", providerName);
> return findOneBy(sc);
> }
>
>
