Loading...
Helix SwarmHelix Swarm
Loading...

Change 13692

norman_morse (Norman Morse)
Norman Morse committed this change 10 years ago into //guest/perforce_software/p4connect/src
Request Review
Download .zip
Fix FileSpec corruption issue

Make sure that the logged in user is verified against files in the default change.
Multiple users can share the same workspace and cause lots of confusion.

Also changed the code in Perforce.P4.Changelist.ToString() to create new FileSpecs instead of doing implicite casts.  This should fix the exception if the
source FileMetaData is incomplete some how.
0 comments Log in to comment
job000288Exception during submit.

NullReferenceException: Object reference not set to... an instance of an object
Perforce.P4.FileMetaData.op_Implicit (Perforce.P4.FileMetaData s) (at
c:/workshop/guest/perforce_software/p4connect/src/P4Bridge/
p4api.net/FileMetaData.cs:866)
Perforce.P4.Changelist.ToString () (at
c:/workshop/guest/perforce_software/p4connect/src/P4Bridge/
p4api.net/Changelist.cs:584)
Perforce.P4.Repository.SaveChangelist (Perforce.P4.Changelist change,
Perforce.P4.Options options) (at
c:/workshop/guest/perforce_software/p4connect/src/P4Bridge/
p4api.net/Repository.Changelist.cs:54)
Perforce.P4.Repository.CreateChangelist (Perforce.P4.Changelist change,
Perforce.P4.Options options) (at
c:/workshop/guest/perforce_software/p4connect/src/P4Bridge/
p4api.net/Repository.Changelist.cs:331)
Perforce.P4.Repository.CreateChangelist (Perforce.P4.Changelist change) (at
c:/workshop/guest/perforce_software/p4connect/src/P4Bridge/
p4api.net/Repository.Changelist.cs:435)
.....

I expect that there is some kind of workspace / user confusion which triggers this problem. «
Add Job
  • Files 5
  • Comments 0
5 edited 0 added 0 deleted
P4Bridge/p4api.net/Changelist.cs#3
Loading...
P4Bridge/p4api.net/FileMetaData.cs#3
Loading...
P4Bridge/p4api.net/FileSpec.cs#3
Loading...
P4Connect/P4Connect/P4Connect.ChangeManager.cs#5
Loading...
P4Connect/P4Connect/P4Connect.Engine.cs#8
Loading...
Tip: Use n and p to cycle through the changes.
Log in to comment