
| Key: |
CCNET-1209
|
| Type: |
Bug
|
| Status: |
Open
|
| Priority: |
Major
|
| Assignee: |
Unassigned
|
| Reporter: |
Kevin Boistis
|
| Votes: |
0
|
| Watchers: |
2
|
|
If you were logged in you would be able to see more operations.
|
|
|
|
|
| Component/s: |
Source Control
|
| Affects Version/s: |
None
|
| Fix Version/s: |
None
|
|
|
Original Estimate:
|
1 hour
|
Remaining Estimate:
|
1 hour
|
Time Spent:
|
Unknown
|
|
|
The GenerateClientView method in ProcessP4Initializer#3.cs has a minor bug which does not allow for quotes in the view. Need to use Quotes for views that contain spaces.
The error is that is inserts at 2 if the view starts with a quote or not:
builder.Append(string.Format(" {0} {1}", viewLine, viewLine.Insert(2, client + "/")));
Updated methos to:
private string GenerateClientView(string[] view, string client)
{
StringBuilder builder = new StringBuilder();
foreach (string viewLine in view)
{
if (viewLine.StartsWith("//"))
builder.Append(string.Format(" {0} {1}", viewLine, viewLine.Insert(2, client + "/")));
else if (viewLine.StartsWith(@"""//"))
builder.Append(string.Format(" {0} {1}", viewLine, viewLine.Insert(3, client + "/")));
builder.Append("\n");
}
return builder.ToString();
}
|
|
Description
|
The GenerateClientView method in ProcessP4Initializer#3.cs has a minor bug which does not allow for quotes in the view. Need to use Quotes for views that contain spaces.
The error is that is inserts at 2 if the view starts with a quote or not:
builder.Append(string.Format(" {0} {1}", viewLine, viewLine.Insert(2, client + "/")));
Updated methos to:
private string GenerateClientView(string[] view, string client)
{
StringBuilder builder = new StringBuilder();
foreach (string viewLine in view)
{
if (viewLine.StartsWith("//"))
builder.Append(string.Format(" {0} {1}", viewLine, viewLine.Insert(2, client + "/")));
else if (viewLine.StartsWith(@"""//"))
builder.Append(string.Format(" {0} {1}", viewLine, viewLine.Insert(3, client + "/")));
builder.Append("\n");
}
return builder.ToString();
} |
Show » |
| There are no comments yet on this issue.
|
|