TRUNK-4411 Relaxed ProviderValidator to accept duplicate identifiers.

- Removed the uniqueness check in the validator.
- Fixed ambiguous java docs (and fixed an error in the documentation).
- The validator only checks for the presence of a retired identifier(with reason for retiring).
- Removed test for unique checks as this behaviour is no longer supported..
- Renamed and fixed tests to accept duplicate entries for Provider identifier.
TRUNK-4411 Relaxed ProviderValidator to accept duplicate identifiers. Updated test names and should annotations.

 - Made them more consistent.
TRUNK-3654: ModuleUtil should not treat SNAPSHOT as 999.

Changed max revision number to Integer.MAX_VALUE and added some test for this change

Added back the lines that stipped off modifiers/qualifiers in the loop

Added back isVersionWithQualifiers() method
added completed date to dialog box
Merge pull request #984 from harsha89/TRUNK-3483-add

added completed date to dialog box
Merge pull request #983 from chocobar/master

TRUNK-4411 Relaxed ProviderValidator to accept duplicate identifiers.
Merge pull request #980 from geoff-wasilwa/TRUNK-3654

TRUNK-3654: ModuleUtil should not treat SNAPSHOT as 999.
TRUNK-4285: Incorrect behavior of Person.getPersonName() and getPersonAddress() for a voided Person

TRUNK-4285: Incorrect behavior of Person.getPersonName() and getPersonAddress() for a voided Person

TRUNK-4285: Incorrect behavior of Person.getPersonName() and getPersonAddress() for a voided Person

TRUNK-4285: Incorrect behavior of Person.getPersonName() and getPersonAddress() for a voided Person
TRUNK-4285: Incorrect behavior of Person.getPersonName() and getPersonAddress() for a voided Person
TRUNK-4285: Incorrect behavior of Person.getPersonName() and getPersonAddress() for a voided Person