job000130 #5

  • //
  • spec/
  • job/
  • job000130
  • View
  • Commits
  • Open Download .zip Download (2 KB)
# The form data below was edited by norman_morse
# Perforce Workshop Jobs
#
#  Job:           The job name. 'new' generates a sequenced job number.
#  Status:        Job status; [open/closed/suspended].  Required
#  Project:       The project this job is for. Required.
#  Severity:      [A/B/C] (A is highest)  Required.
#  ReportedBy     The user who created the job. Can be changed.
#  ReportedDate:  The date the job was created.  Automatic.
#  ModifiedBy:    The user who last modified this job. Automatic.
#  ModifiedDate:  The date this job was last modified. Automatic.
#  OwnedBy:       The owner, responsible for doing the job. Optional.
#  Description:   Description of the job.  Required.
#  DevNotes:      Developer's comments.  Optional.
#  Type:	  Type of job; [Bug/Feature].  Required.

Job:	job000130

Status:	closed

Project:	perforce-software-p4connect

Severity:	B

ReportedBy:	jan_van_uytven

ReportedDate:	2014/12/12 15:31:45

ModifiedBy:	norman_morse

ModifiedDate:	2015/10/12 17:03:23

Description:
	Right-clicking on Assets and selecting Perforce-> Add to Depot results in no files being added.
	
	The Pending changelist is empty, and no error is reported.  Instead, you are required to add each subfolder separately.
	
	No recursive add/edit/delete
	
	Unity Asset Store user Jordan:
	    > "... the most glaring issue I see right off the bat is the lack of recusive  add/edit. I
	    > expected the behaviour in the plugin to match P4V, where an action on a folder also
	    > does that action on all files under that folder."
	    > 
	    > "From what I can tell, the GetDeepSelectedEffectivePaths call uses
	    > Selection.GetFiltered(typeof(Object), SelectionMode.Assets), which should actually be
	    > SelectionMode.DeepAssets instead."
	    > 
	    > "So far so good, but I'd like to see behaviour match closer to P4 (command line) and
	    > P4V. 

DevNotes:
	Yes, P4Connect needs to handle recursive calls with "Add to Depot", and other commands.
	
	Fixed in 2015.2 release.

Type:	Bug
# Change User Description Committed
#5 default
#4 default
#3 default
#2 default
#1 default