TNO-2566: Evening Overview Error #1796
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,119 @@ | ||
| DO $$ | ||
| BEGIN | ||
|
|
||
| UPDATE public.report_template SET | ||
| "body" = ' | ||
| @inherits RazorEngineCore.RazorEngineTemplateBase<TNO.Services.Reporting.Models.ReportEngineAVOverviewModel> | ||
| @using System | ||
| @using System.Linq | ||
| @using TNO.Entities | ||
| <style> | ||
| a { text-decoration:none; } | ||
| .video-icon { fill: #702828; height: 20px; width: 20px; } | ||
| .transcript-icon { fill: #047A8C; height: 20px; width: 20px;} | ||
| .icon { float: right; } | ||
| .main-body { | ||
| width: 595px; | ||
| } | ||
| </style> | ||
| <div class="main-body"> | ||
| <div><p style="background-color: #FFF7E1; color: #876503; text-align: center; font-size: 16px; font-weight: 700; line-height: 110%; letter-spacing: 0.08px; padding: 10px 0px; margin: 6px 0px 20px 0px;">This TNO product is intended only for the use of the person to whom it is addressed. Please do not forward or redistribute.</p></div> | ||
| <div style="padding-top: 10px; border-radius:25px; background-color:rgba(255,255,255, .25)"> | ||
| <div style="padding-bottom:20px;"><img src="https://dev.mmi.gov.bc.ca/assets/MMinsights_logo_black.svg" width="500px"></div> | ||
| </div> | ||
| <h1 id="top" style="color: #971D29; font-size: 32px; line-height: 38px; margin: 0px; border-bottom: 1px solid #6C5D62;">Evening Overview</h1> | ||
|
|
||
|
|
||
| <div style="font-size:1.12rem; font-weight:600;"><p>@($"{Instance.PublishedOn:dddd, MMMM dd, yyyy}")</p></div> | ||
|
|
||
| <br/> | ||
| @if (Instance.Sections.Count() == 0) | ||
| { | ||
| <p>There is no content in this report.</p> | ||
| } | ||
| @foreach (var section in Instance.Sections) | ||
| { | ||
| var itemIndex = 0; | ||
| var itemCount = section.Items.Count(i => i.ItemType == AVOverviewItemType.Story); | ||
|
|
||
| <div class="section" style="border: 1px solid #9F9196; border-radius: 6px; margin-bottom:20px; padding: 16px;"> | ||
|
|
||
| <div class="section-header" style="text-align:center; font-size:1.1rem; font-weight:600; padding-bottom:6px;"> | ||
| @section.Name @(section.StartTime?.Length == 8 ? section.StartTime.Substring(0, 5) : section.StartTime) @(section.StartTime?.Length > 0 ? "hr" : "") | ||
| @if (!String.IsNullOrEmpty(section.Anchors)) { <span> - @section.Anchors</span> } | ||
| </div> | ||
|
|
||
|
|
||
| <table style="width:100%; border: collapse;"> | ||
| <thead style="background-color:#9F9196;"> | ||
| <tr> | ||
| <td style="text-transform:uppercase; font-weight:500; color:#FFF; padding:0 10px; width:8rem;">Placement</td> | ||
| <td style="text-transform:uppercase; font-weight:500; color:#FFF; padding:0 10px; width:6rem;">Time</td> | ||
| <td style="text-transform:uppercase; font-weight:500; color:#FFF; padding:0 10px;">Story</td> | ||
| </tr> | ||
| </thead> | ||
|
|
||
|
|
||
| @foreach (var item in section.Items) | ||
| { | ||
| var placement = item.ItemType switch | ||
| { | ||
| AVOverviewItemType.Intro => "Intro", | ||
| AVOverviewItemType.Story => itemIndex == 1 ? "Top Story" : $"{itemIndex}/{itemCount}", | ||
| AVOverviewItemType.Ad => "", | ||
| _ => $"{item.ItemType}", | ||
| }; | ||
| var time = item.ItemType switch | ||
| { | ||
| AVOverviewItemType.Ad => "", | ||
| _ => $"{item.Time}", | ||
| }; | ||
| var summary = String.IsNullOrEmpty(item.Summary) ? "---" : item.Summary.Replace("\n", "<br/>"); | ||
| <tr style="border-bottom: 1px solid #DED9DA;"> | ||
| <td style="border-bottom: 1px solid #DED9DA; font-size:.9rem; padding:2px 2px 2px 10px;">@placement</td> | ||
| <td style="border-bottom: 1px solid #DED9DA; font-size:.9rem; padding:2px 2px 2px 10px;">@(time?.Length == 8 ? time.Substring(0, 5) : time)</td> | ||
| <td style="border-bottom: 1px solid #DED9DA; font-size:.9rem; padding:2px 2px 2px 10px;"> | ||
| @if (item.ContentId.HasValue) | ||
| { | ||
| <a href="@($"{ViewContentUrl}{item.ContentId}")">@summary</a> | ||
| @if (item.Content.FileReferences != null && item.Content.FileReferences.Any() && item.Content.FileReferences.FirstOrDefault().ContentType.StartsWith("video/") ) | ||
| { | ||
| <a class="icon" href="@($"{ViewContentUrl}{item.ContentId}")"> | ||
| <svg class="video-icon" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 576 512"><path d="M0 128C0 92.7 28.7 64 64 64H320c35.3 0 64 28.7 64 64V384c0 35.3-28.7 64-64 64H64c-35.3 0-64-28.7-64-64V128zM559.1 99.8c10.4 5.6 16.9 16.4 16.9 28.2V384c0 11.8-6.5 22.6-16.9 28.2s-23 5-32.9-1.6l-96-64L416 337.1V320 192 174.9l14.2-9.5 96-64c9.8-6.5 22.4-7.2 32.9-1.6z"/></svg> | ||
| </a> | ||
| } | ||
| @if (item.Content.ContentType.ToString() == "AudioVideo" && item.Content.Summary.Length > 0 && item.Content.IsApproved) | ||
| { | ||
| <a class="icon" href="@($"{ViewContentUrl}{item.ContentId}")"> | ||
| <svg style="float: right" class="transcript-icon" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 512 512"><path d="M278.5 215.6L23 471c-9.4 9.4-9.4 24.6 0 33.9s24.6 9.4 33.9 0l57-57h68c49.7 0 97.9-14.4 139-41c11.1-7.2 5.5-23-7.8-23c-5.1 0-9.2-4.1-9.2-9.2c0-4.1 2.7-7.6 6.5-8.8l81-24.3c2.5-.8 4.8-2.1 6.7-4l22.4-22.4c10.1-10.1 2.9-27.3-11.3-27.3l-32.2 0c-5.1 0-9.2-4.1-9.2-9.2c0-4.1 2.7-7.6 6.5-8.8l112-33.6c4-1.2 7.4-3.9 9.3-7.7C506.4 207.6 512 184.1 512 160c0-41-16.3-80.3-45.3-109.3l-5.5-5.5C432.3 16.3 393 0 352 0s-80.3 16.3-109.3 45.3L139 149C91 197 64 262.1 64 330v55.3L253.6 195.8c6.2-6.2 16.4-6.2 22.6 0c5.4 5.4 6.1 13.6 2.2 19.8z"/></svg> | ||
| </a> | ||
| } | ||
| } | ||
| else | ||
| { | ||
| @summary | ||
| } | ||
| </td> | ||
| </tr> | ||
| @{ | ||
| itemIndex++; | ||
| } | ||
| } | ||
| </table> | ||
| </div> | ||
| } | ||
| @* FOOTER *@ | ||
|
|
||
| <div style="margin-top:20px; background-color:#F5F6F9; color:#6C5D62; text-align: center; font-size: 11px; font-style: normal; font-weight: 500; line-height: 110%; letter-spacing: 0.08px; padding: 10px 0; width:100%;"> | ||
| <p>Terms of Use</p> | ||
| <p> This summary is a service provided by Government Communications and Public Engagement and is only intended for original addressee. All content is the copyrighted property of a third party creator of the material.<br /><br /> | ||
| <b>Copying, retransmitting, archiving, redistributing, selling, licensing, or emailing the material to any third party or any employee of the Province who is not authorized to access the material is prohibited.</b> | ||
| </p> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| ' | ||
|
|
||
| WHERE "name" = 'AV Overview - Weekday'; | ||
|
|
||
| END $$; | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,119 @@ | ||
| DO $$ | ||
| BEGIN | ||
|
|
||
| UPDATE public.report_template SET | ||
| "body" = ' | ||
| @inherits RazorEngineCore.RazorEngineTemplateBase<TNO.Services.Reporting.Models.ReportEngineAVOverviewModel> | ||
| @using System | ||
| @using System.Linq | ||
| @using TNO.Entities | ||
| <style> | ||
| a { text-decoration:none; } | ||
| .video-icon { fill: #702828; height: 20px; width: 20px; } | ||
| .transcript-icon { fill: #047A8C; height: 20px; width: 20px;} | ||
| .icon { float: right; } | ||
| .main-body { | ||
| width: 595px; | ||
| } | ||
| </style> | ||
| <div class="main-body"> | ||
| <div><p style="background-color: #FFF7E1; color: #876503; text-align: center; font-size: 16px; font-weight: 700; line-height: 110%; letter-spacing: 0.08px; padding: 10px 0px; margin: 6px 0px 20px 0px;">This TNO product is intended only for the use of the person to whom it is addressed. Please do not forward or redistribute.</p></div> | ||
| <div style="padding-top: 10px; border-radius:25px; background-color:rgba(255,255,255, .25)"> | ||
| <div style="padding-bottom:20px;"><img src="https://dev.mmi.gov.bc.ca/assets/MMinsights_logo_black.svg" width="500px"></div> | ||
| </div> | ||
| <h1 id="top" style="color: #971D29; font-size: 32px; line-height: 38px; margin: 0px; border-bottom: 1px solid #6C5D62;">Evening Overview</h1> | ||
|
|
||
|
|
||
| <div style="font-size:1.12rem; font-weight:600;"><p>@($"{Instance.PublishedOn:dddd, MMMM dd, yyyy}")</p></div> | ||
|
|
||
| <br/> | ||
| @if (Instance.Sections.Count() == 0) | ||
| { | ||
| <p>There is no content in this report.</p> | ||
| } | ||
| @foreach (var section in Instance.Sections) | ||
| { | ||
| var itemIndex = 0; | ||
| var itemCount = section.Items.Count(i => i.ItemType == AVOverviewItemType.Story); | ||
|
|
||
| <div class="section" style="border: 1px solid #9F9196; border-radius: 6px; margin-bottom:20px; padding: 16px;"> | ||
|
|
||
| <div class="section-header" style="text-align:center; font-size:1.1rem; font-weight:600; padding-bottom:6px;"> | ||
| @section.Name @(section.StartTime?.Length == 8 ? section.StartTime.Substring(0, 5) : section.StartTime) @(section.StartTime?.Length > 0 ? "hr" : "") | ||
| @if (!String.IsNullOrEmpty(section.Anchors)) { <span> - @section.Anchors</span> } | ||
| </div> | ||
|
|
||
|
|
||
| <table style="width:100%; border: collapse;"> | ||
| <thead style="background-color:#9F9196;"> | ||
| <tr> | ||
| <td style="text-transform:uppercase; font-weight:500; color:#FFF; padding:0 10px; width:8rem;">Placement</td> | ||
| <td style="text-transform:uppercase; font-weight:500; color:#FFF; padding:0 10px; width:6rem;">Time</td> | ||
| <td style="text-transform:uppercase; font-weight:500; color:#FFF; padding:0 10px;">Story</td> | ||
| </tr> | ||
| </thead> | ||
|
|
||
|
|
||
| @foreach (var item in section.Items) | ||
| { | ||
| var placement = item.ItemType switch | ||
| { | ||
| AVOverviewItemType.Intro => "Intro", | ||
| AVOverviewItemType.Story => itemIndex == 1 ? "Top Story" : $"{itemIndex}/{itemCount}", | ||
| AVOverviewItemType.Ad => "", | ||
| _ => $"{item.ItemType}", | ||
| }; | ||
| var time = item.ItemType switch | ||
| { | ||
| AVOverviewItemType.Ad => "", | ||
| _ => $"{item.Time}", | ||
| }; | ||
| var summary = String.IsNullOrEmpty(item.Summary) ? "---" : item.Summary.Replace("\n", "<br/>"); | ||
| <tr style="border-bottom: 1px solid #DED9DA;"> | ||
| <td style="border-bottom: 1px solid #DED9DA; font-size:.9rem; padding:2px 2px 2px 10px;">@placement</td> | ||
| <td style="border-bottom: 1px solid #DED9DA; font-size:.9rem; padding:2px 2px 2px 10px;">@(time?.Length == 8 ? time.Substring(0, 5) : time)</td> | ||
| <td style="border-bottom: 1px solid #DED9DA; font-size:.9rem; padding:2px 2px 2px 10px;"> | ||
| @if (item.ContentId.HasValue) | ||
| { | ||
| <a href="@($"{ViewContentUrl}{item.ContentId}")">@summary</a> | ||
| @if (item.Content.FileReferences != null && item.Content.FileReferences.Any() && item.Content.FileReferences.FirstOrDefault().ContentType.StartsWith("video/") ) | ||
|
There was a problem hiding this comment.
There was a problem hiding this comment. Added Please let me know if I should check for another way to implement it. Thank you! |
||
| { | ||
| <a class="icon" href="@($"{ViewContentUrl}{item.ContentId}")"> | ||
| <svg class="video-icon" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 576 512"><path d="M0 128C0 92.7 28.7 64 64 64H320c35.3 0 64 28.7 64 64V384c0 35.3-28.7 64-64 64H64c-35.3 0-64-28.7-64-64V128zM559.1 99.8c10.4 5.6 16.9 16.4 16.9 28.2V384c0 11.8-6.5 22.6-16.9 28.2s-23 5-32.9-1.6l-96-64L416 337.1V320 192 174.9l14.2-9.5 96-64c9.8-6.5 22.4-7.2 32.9-1.6z"/></svg> | ||
| </a> | ||
| } | ||
| @if (item.Content.ContentType.ToString() == "AudioVideo" && item.Content.Summary.Length > 0 && item.Content.IsApproved) | ||
| { | ||
| <a class="icon" href="@($"{ViewContentUrl}{item.ContentId}")"> | ||
| <svg style="float: right" class="transcript-icon" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 512 512"><path d="M278.5 215.6L23 471c-9.4 9.4-9.4 24.6 0 33.9s24.6 9.4 33.9 0l57-57h68c49.7 0 97.9-14.4 139-41c11.1-7.2 5.5-23-7.8-23c-5.1 0-9.2-4.1-9.2-9.2c0-4.1 2.7-7.6 6.5-8.8l81-24.3c2.5-.8 4.8-2.1 6.7-4l22.4-22.4c10.1-10.1 2.9-27.3-11.3-27.3l-32.2 0c-5.1 0-9.2-4.1-9.2-9.2c0-4.1 2.7-7.6 6.5-8.8l112-33.6c4-1.2 7.4-3.9 9.3-7.7C506.4 207.6 512 184.1 512 160c0-41-16.3-80.3-45.3-109.3l-5.5-5.5C432.3 16.3 393 0 352 0s-80.3 16.3-109.3 45.3L139 149C91 197 64 262.1 64 330v55.3L253.6 195.8c6.2-6.2 16.4-6.2 22.6 0c5.4 5.4 6.1 13.6 2.2 19.8z"/></svg> | ||
| </a> | ||
| } | ||
| } | ||
| else | ||
| { | ||
| @summary | ||
| } | ||
| </td> | ||
| </tr> | ||
| @{ | ||
| itemIndex++; | ||
| } | ||
| } | ||
| </table> | ||
| </div> | ||
| } | ||
| @* FOOTER *@ | ||
|
|
||
| <div style="margin-top:20px; background-color:#F5F6F9; color:#6C5D62; text-align: center; font-size: 11px; font-style: normal; font-weight: 500; line-height: 110%; letter-spacing: 0.08px; padding: 10px 0; width:100%;"> | ||
| <p>Terms of Use</p> | ||
| <p> This summary is a service provided by Government Communications and Public Engagement and is only intended for original addressee. All content is the copyrighted property of a third party creator of the material.<br /><br /> | ||
| <b>Copying, retransmitting, archiving, redistributing, selling, licensing, or emailing the material to any third party or any employee of the Province who is not authorized to access the material is prohibited.</b> | ||
| </p> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| ' | ||
|
|
||
| WHERE "name" = 'AV Overview - Weekday'; | ||
|
|
||
| END $$; | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.FirstOrDefault()can result in a null value. This will fail if there is a null value.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added
item.Content.FileReferences.Any()to check if there is any content and just in that case obtain the first record with contentType starting with 'video'. Ifitem.Content.FileReferences.Any()returns false then it does not proceed to the rest of the conditional if statement, so it would not executeFirstOrDefault().Please let me know if I should check for another way to implement it.
Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this will work. You can also do
item.Content.FileReferences.FirstOrDefault()?.ContentType.StartsWith("video/") == trueThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!