Skip to content

Commit

Permalink
doing more fixes before production
Browse files Browse the repository at this point in the history
  • Loading branch information
jgauffin committed Jun 16, 2016
1 parent 2d80516 commit 3714060
Show file tree
Hide file tree
Showing 38 changed files with 31 additions and 51 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -32,3 +32,4 @@ src/*/.vs/*
src/*/packages/*/
src/Tools/MarkdownToNamespaceDoc/packages/*/
src/Tools/TsGenerator/.vs/*
src/Help/*
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public void should_store_settings_with_dot_notation()
{
var expected = "World" + Guid.NewGuid().ToString("N");
var category = new WriteTestSection();
category.Items["Hello"] = expected;
category.Properties["Hello"] = expected;

var sut = new ConfigFileStore();
sut.Store(category);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@ internal class WriteTestSection : IConfigurationSection
{
public WriteTestSection()
{
Items = new Dictionary<string, string>();
Properties = new Dictionary<string, string>();
}

public IDictionary<string, string> Items { get; private set; }
public IDictionary<string, string> Properties { get; private set; }

public string SectionName
{
Expand All @@ -20,12 +20,12 @@ public string SectionName

public IDictionary<string, string> ToDictionary()
{
return Items;
return Properties;
}

public void Load(IDictionary<string, string> settings)
{
Items = settings;
Properties = settings;
}
}
}
1 change: 0 additions & 1 deletion src/Server/OneTrueError.App/OneTrueError.App.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,6 @@
<Compile Include="Core\Invitations\Invitation.cs" />
<Compile Include="Core\Notifications\Tasks\SendIncidentEmail.cs" />
<Compile Include="Core\Notifications\Tasks\SendIncidentSms.cs" />
<Compile Include="Modules\Messaging\Commands\PostMarkSendEmailHandler.cs" />
<Compile Include="Modules\Messaging\ITemplateParser.cs" />
<Compile Include="Modules\Messaging\Template.cs" />
<Compile Include="Modules\Messaging\Templating\DateFormatter.cs" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,21 +21,21 @@ protected ReceivedReportContextInfo()
/// Creates a new instance of <see cref="ReceivedReportContextInfo" />.
/// </summary>
/// <param name="name">context collection name</param>
/// <param name="items">properties</param>
/// <param name="properties">properties</param>
/// <exception cref="ArgumentNullException">name; items</exception>
public ReceivedReportContextInfo(string name, Dictionary<string, string> items)
public ReceivedReportContextInfo(string name, Dictionary<string, string> properties)
{
if (name == null) throw new ArgumentNullException(nameof(name));
if (items == null) throw new ArgumentNullException(nameof(items));
if (name == null) throw new ArgumentNullException("name");
if (properties == null) throw new ArgumentNullException("properties");

Name = name;
Items = items;
Properties = properties;
}

/// <summary>
/// Properties
/// </summary>
public Dictionary<string, string> Items { get; set; }
public Dictionary<string, string> Properties { get; set; }


/// <summary>
Expand All @@ -53,7 +53,7 @@ public ReceivedReportContextInfo(string name, Dictionary<string, string> items)
public override string ToString()
{
return Name + " [" + string.Join(", ",
Items.Select(x => x.Key + "=" + x.Value)) + "]";
Properties.Select(x => x.Key + "=" + x.Value)) + "]";
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ public ErrorReportEntity ConvertReport(ReceivedReportDTO report, int application
}

//var id = _idGeneratorClient.GetNextId(ErrorReportEntity.SEQUENCE);
var contexts = report.ContextCollections.Select(x => new ErrorReportContext(x.Name, x.Items)).ToArray();
var contexts = report.ContextCollections.Select(x => new ErrorReportContext(x.Name, x.Properties)).ToArray();
var dto = new ErrorReportEntity(applicationId, report.ReportId, report.CreatedAtUtc, ex, contexts);
return dto;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ public bool Execute()

private ErrorReportContext ConvertContext(ReceivedReportContextInfo arg)
{
return new ErrorReportContext(arg.Name, arg.Items);
return new ErrorReportContext(arg.Name, arg.Properties);
}

private ErrorReportException ConvertException(ReceivedReportException dto)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -104,13 +104,13 @@ private static ReceivedReportException ConvertException(NewReportException excep
StackTrace = exception.StackTrace
};
if (exception.InnerException != null)
ex.InnerException = ConvertException(exception);
ex.InnerException = ConvertException(exception.InnerException);
return ex;
}

private static ReceivedReportContextInfo ConvertCollection(NewReportContextInfo arg)
{
return new ReceivedReportContextInfo(arg.Name, arg.Items);
return new ReceivedReportContextInfo(arg.Name, arg.Properties);
}

private NewReportDTO DeserializeBody(byte[] body)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,21 +9,21 @@ public NewReportContextInfo()
{
}

public NewReportContextInfo(string name, Dictionary<string, string> items)
public NewReportContextInfo(string name, Dictionary<string, string> properties)
{
Name = name;
Items = items;
Properties = properties;
}


public string Name { get; set; }

public Dictionary<string, string> Items { get; set; }
public Dictionary<string, string> Properties { get; set; }

public override string ToString()
{
return Name + " [" + string.Join(", ",
Items.Select(x => x.Key + "=" + x.Value)) + "]";
Properties.Select(x => x.Key + "=" + x.Value)) + "]";
}
}
}
1 change: 0 additions & 1 deletion src/Server/OneTrueError.Web/Scripts/CqsClient.js
Original file line number Diff line number Diff line change
Expand Up @@ -56,4 +56,3 @@ var Griffin;
Cqs.CqsClient = CqsClient;
})(Cqs = Griffin.Cqs || (Griffin.Cqs = {}));
})(Griffin || (Griffin = {}));
//# sourceMappingURL=CqsClient.js.map
1 change: 0 additions & 1 deletion src/Server/OneTrueError.Web/Scripts/Griffin.Editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -645,4 +645,3 @@ var Griffin;
}());
Griffin.TextSelector = TextSelector;
})(Griffin || (Griffin = {}));
//# sourceMappingURL=Griffin.Editor.js.map
1 change: 0 additions & 1 deletion src/Server/OneTrueError.Web/Scripts/Griffin.Net.js
Original file line number Diff line number Diff line change
Expand Up @@ -224,4 +224,3 @@ var Griffin;
Net.HttpClient = HttpClient;
})(Net = Griffin.Net || (Griffin.Net = {}));
})(Griffin || (Griffin = {})); // ReSharper restore InconsistentNaming
//# sourceMappingURL=Griffin.Net.js.map
1 change: 0 additions & 1 deletion src/Server/OneTrueError.Web/Scripts/Griffin.WebApp.js
Original file line number Diff line number Diff line change
Expand Up @@ -364,4 +364,3 @@ var Griffin;
WebApp.Pager = Pager;
})(WebApp = Griffin.WebApp || (Griffin.WebApp = {}));
})(Griffin || (Griffin = {}));
//# sourceMappingURL=Griffin.WebApp.js.map
1 change: 0 additions & 1 deletion src/Server/OneTrueError.Web/Scripts/Models/AllModels.js
Original file line number Diff line number Diff line change
Expand Up @@ -1394,4 +1394,3 @@ var OneTrueError;
})(Accounts = Core.Accounts || (Core.Accounts = {}));
})(Core = OneTrueError.Core || (OneTrueError.Core = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=AllModels.js.map
5 changes: 2 additions & 3 deletions src/Server/OneTrueError.Web/Scripts/Promise.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/**
Module P: Generic Promises for TypeScript
Module P: Generic Promises for TypeScript
Project, documentation, and license: https://github.com/pragmatrix/Promise
Project, documentation, and license: https://github.com/pragmatrix/Promise
*/
var P;
(function (P) {
Expand Down Expand Up @@ -321,4 +321,3 @@ var P;
}
P.isUndefined = isUndefined;
})(P || (P = {}));
//# sourceMappingURL=Promise.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,3 @@ var OneTrueError;
Account.AcceptedViewModel = AcceptedViewModel;
})(Account = OneTrueError.Account || (OneTrueError.Account = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=AcceptedViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -54,4 +54,3 @@ var OneTrueError;
Account.SettingsViewModel = SettingsViewModel;
})(Account = OneTrueError.Account || (OneTrueError.Account = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=SettingsViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,3 @@ var OneTrueError;
Account.WelcomeViewModel = WelcomeViewModel;
})(Account = OneTrueError.Account || (OneTrueError.Account = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=WelcomeViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -279,4 +279,3 @@ var OneTrueError;
Application.DetailsViewModel = DetailsViewModel;
})(Application = OneTrueError.Application || (OneTrueError.Application = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=DetailsViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,3 @@ var OneTrueError;
Application.InstallationViewModel = InstallationViewModel;
})(Application = OneTrueError.Application || (OneTrueError.Application = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=InstallationViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -45,4 +45,3 @@ var OneTrueError;
Application.TeamViewModel = TeamViewModel;
})(Application = OneTrueError.Application || (OneTrueError.Application = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=TeamViewModel.js.map
1 change: 0 additions & 1 deletion src/Server/OneTrueError.Web/ViewModels/ChartViewModel.js
Original file line number Diff line number Diff line change
Expand Up @@ -142,4 +142,3 @@ var OneTrueError;
}());
OneTrueError.LineChart = LineChart;
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=ChartViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -72,4 +72,3 @@ var OneTrueError;
Feedback.ApplicationViewModel = ApplicationViewModel;
})(Feedback = OneTrueError.Feedback || (OneTrueError.Feedback = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=ApplicationViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -61,4 +61,3 @@ var OneTrueError;
Feedback.IncidentViewModel = IncidentViewModel;
})(Feedback = OneTrueError.Feedback || (OneTrueError.Feedback = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=IncidentViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -62,4 +62,3 @@ var OneTrueError;
Feedback.OverviewViewModel = OverviewViewModel;
})(Feedback = OneTrueError.Feedback || (OneTrueError.Feedback = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=OverviewViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,3 @@ var OneTrueError;
Home.WelcomeViewModel = WelcomeViewModel;
})(Home = OneTrueError.Home || (OneTrueError.Home = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=WelcomeViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -66,4 +66,3 @@ var OneTrueError;
Incident.CloseViewModel = CloseViewModel;
})(Incident = OneTrueError.Incident || (OneTrueError.Incident = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=CloseViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -46,4 +46,3 @@ var OneTrueError;
Incident.IgnoreViewModel = IgnoreViewModel;
})(Incident = OneTrueError.Incident || (OneTrueError.Incident = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=IgnoreViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -161,4 +161,3 @@ var OneTrueError;
Incident.IncidentViewModel = IncidentViewModel;
})(Incident = OneTrueError.Incident || (OneTrueError.Incident = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=IncidentViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -162,4 +162,3 @@ var OneTrueError;
Incident.IndexViewModel = IndexViewModel;
})(Incident = OneTrueError.Incident || (OneTrueError.Incident = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=IndexViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -62,4 +62,3 @@ var OneTrueError;
Incident.OriginsViewModel = OriginsViewModel;
})(Incident = OneTrueError.Incident || (OneTrueError.Incident = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=OriginsViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -102,4 +102,3 @@ var OneTrueError;
Incident.SimilaritiesViewModel = SimilaritiesViewModel;
})(Incident = OneTrueError.Incident || (OneTrueError.Incident = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=SimilaritiesViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -242,4 +242,3 @@ var OneTrueError;
Overview.OverviewViewModel = OverviewViewModel;
})(Overview = OneTrueError.Overview || (OneTrueError.Overview = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=OverviewViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,13 @@ var OneTrueError;
var target = evt.target;
if (target.tagName === 'LI') {
_this.selectCollection(target.firstElementChild.textContent);
$('li', target.parentElement).removeClass('active');
$(target).addClass('active');
}
else if (target.tagName === 'A') {
_this.selectCollection(target.textContent);
$('li', target.parentElement.parentElement).removeClass('active');
$(target.parentElement).addClass('active');
}
}, true);
};
Expand All @@ -77,4 +81,3 @@ var OneTrueError;
Report.ReportViewModel = ReportViewModel;
})(Report = OneTrueError.Report || (OneTrueError.Report = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=ReportViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,12 @@ module OneTrueError.Report {
var target = <HTMLElement>evt.target;
if (target.tagName === 'LI') {
this.selectCollection(target.firstElementChild.textContent);
$('li', target.parentElement).removeClass('active');
$(target).addClass('active');
} else if (target.tagName === 'A') {
this.selectCollection(target.textContent);
$('li', target.parentElement.parentElement).removeClass('active');
$(target.parentElement).addClass('active');
}
}, true);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,4 +40,3 @@ var OneTrueError;
User.NotificationsViewModel = NotificationsViewModel;
})(User = OneTrueError.User || (OneTrueError.User = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=NotificationsViewModel.js.map
Original file line number Diff line number Diff line change
Expand Up @@ -62,4 +62,3 @@ var OneTrueError;
User.SettingsViewModel = SettingsViewModel;
})(User = OneTrueError.User || (OneTrueError.User = {}));
})(OneTrueError || (OneTrueError = {}));
//# sourceMappingURL=SettingsViewModel.js.map
1 change: 0 additions & 1 deletion src/Server/OneTrueError.Web/app/Application.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,4 +33,3 @@ var OneTrueError;
})(Applications = OneTrueError.Applications || (OneTrueError.Applications = {}));
})(OneTrueError || (OneTrueError = {}));
;
//# sourceMappingURL=Application.js.map

0 comments on commit 3714060

Please sign in to comment.