CLOUDSTACK-4736 Fixed issue in default service selection
Fix CID 1127027 Dereference before null check

template can't be null at this point as it has already been dereferenced
Fix CID 1127021 Operands don't affect results

int can never be bigger than maxint.

Auto unboxing of Integer, not need for intValue()
Fix CID 1127045 Dead local store
Fix CID 1127041,1127040, 1127039, 1127038, 1127037 String concatenation in loop
Fix CID 1127024 Various resource leaks
Fix CID 1127030, 1127029 Questionable Boxing of primitive value
Fix CID 1127028 Fix Inefficient Map Iterator
Fix bug.
Fix CID 1127031 Inefficient Map iterator
Fix CID 1127051 DLS: Dead local store
Fix CID 1127050 WMI: Inefficient Map iterator
CLOUDSTACK-4793 : Added UpgradeRouterTemplate API. Added filters to listRouters API. listRouters response includes verion and required upgrade flag. Min VR version is checked before sending commands to router
Fix CID 1116741,1116742 Resource leak
Fix CID 1116743 Resource leak
CLOUDSTACK-5039 bugfix: live migration failure in KVM env.

Signed-off-by: ynojima <mail@ynojima.net>
Fix Palo Alto Firewall RAT issue
CLOUDSTACK-4793: UI > Infrastructure > Virtual Routers > listView > implement Advanced Search by name/zone/pod/domain/account.
fixed Rules

it was no longer working, it must be fixed for those who want to read it.

Signed-off-by: Laszlo Hornyak <laszlo.hornyak@gmail.com>
Removing coded documentation

This is the output of the program, the developer documentation may be a better place

-----

When developing against the CloudStack Orchestration Platform, you must following the following rules:
API Rule: Always be prepared to handle RuntimeExceptions.

When writing APIs, you must follow these rules:
API Rule: You may think you're the greatest developer in the world but every change to the API must be reviewed and approved.
API Rule: Every API must have unit tests written against it.  And not it's unit tests
API Rule:

-----

Signed-off-by: Laszlo Hornyak <laszlo.hornyak@gmail.com>
CLOUDSTACK-4058: UI > Host detail page > (1) Add Hypervisor, HypervisorVersion field.
(2) Remove CloudstackVersion field.
FileUtil: use commons-io FileUtils

- removed the method body ot FileUtil.copyFile and replaced with FleUtils.copyFile

Signed-off-by: Laszlo Hornyak <laszlo.hornyak@gmail.com>
CLOUDSTACK-5089: fixed VPCDao to set the accountId field correctly
Added option to reload VM during in RevertToVMSnapshotCommand (only in VMWare as I don't believe that XenServer supports anything similar)

Signed-off-by: Edison Su <sudison@gmail.com>