DL API to move file includes mandatory field in serviceContext

• Type: Bug
• Status: In Review
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 6.1.1 CE GA2, 6.1.20 EE GA2
• Fix Version/s: --Sprint 11/12, 6.2.0 CE M2
• Component/s: API, API > Portal Service, DM > Document Library Display
• Labels: None
• Fix Priority: 3
• Where was the bug?:
  Java
• Similar Issues: Show 5 results ; ;

  LPS-7923	Include Move command for EditFileEntryAction.java
  LPS-1955	Mandatory fields in web content management structure
  LPS-7514	Let the user know which fields are mandatory in forms (aui forms: mandatory)
  LPS-15524	Web Form Portlet - Custom Field does not generate mandatory field error
  LPS-34490	Images displayed using DDM don't have alt attribute (mandatory in html)

DL service calls to move file entry requires serviceContext.scopeGroupId to be filled if moving to default parent folder. Mandatory fields should not be in serviceContext.

	protected Repository getRepository(
			long folderId, ServiceContext serviceContext)
		throws PortalException, SystemException {

		Repository repository = null;

		if (folderId == DLFolderConstants.DEFAULT_PARENT_FOLDER_ID) {
			repository = getRepository(serviceContext.getScopeGroupId());
		}
		else {
			repository = getRepository(folderId, 0, 0);
		}

		return repository;
	}

[Igor Spasic]

serviceContext.scopeGroupId is still needed for further execution (used by permission checker, and maybe later as well), so I am setting it up in the code when target folder is 0 (default)

[Alexander Chow]

Updated the pull request to clean up scopeGroupId in DL