8000 Added Meta, Charset, and Transitional parameters to ConvertTo-HTML by ergo3114 · Pull Request #4184 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]
Skip to content

Added Meta, Charset, and Transitional parameters to ConvertTo-HTML #4184

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 31 commits into from
Sep 11, 2017
Merged

Added Meta, Charset, and Transitional parameters to ConvertTo-HTML #4184

Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
be4d32c
- Added functionality to convert-HTML.cs to include a charset parameter
ergo3114 Jun 28, 2017
883df45
- Fixed issues on the charset parameter where flag was always set to …
ergo3114 Jun 29, 2017
3caab36
- Moved Meta tag into Head tag
ergo3114 Jun 29, 2017
cdb3535
- Took away charset defaullt value
ergo3114 Jun 30, 2017
d0222c5
- Added Transitional switch to allow users to switch between strict a…
ergo3114 Jul 4, 2017
35b44c0
- Added validation to the Charset parameter
ergo3114 Jul 15, 2017
6b325b7
- Put the Meta parameter output inside the HTML head tag
ergo3114 Jul 15, 2017
7d39a6c
- Added filtering for the Meta parameter to only allow defined HTML m…
ergo3114 Jul 16, 2017
ac28b95
- Changed the condition for http-equiv from && to ||
ergo3114 Jul 16, 2017
3baca91
- Removed static declaration while attempting to using InvocationInfo…
ergo3114 Jul 16, 2017
56a7db3
- Removed defined get/set functions from the transitional switch para…
ergo3114 Jul 22, 2017
71168cc
- Removed commented out testing
ergo3114 Jul 22, 2017
06ef847
Merge remote-tracking branch 'refs/remotes/PowerShell/master'
ergo3114 Jul 23, 2017
cbec470
- Removed \1 in the root folder
ergo3114 Jul 24, 2017
3daa019
- Added pattern validation to the charset parameter
ergo3114 Jul 25, 2017
0f9279b
- Added error message in resource file
ergo3114 Jul 29, 2017
b6cdea7
- Fixed error introduced in test on push 0f9279b when testing meta pa…
ergo3114 Jul 29, 2017
06e9751
- Removed refresh as a valid meta property
ergo3114 Jul 30, 2017
d77d17c
- Updated Meta parameter validation to a switch statement from an if …
ergo3114 Jul 31, 2017
bd1b6ca
- Removed comments
ergo3114 Aug 5, 2017
65ef76c
- Added X-UA-Compatible as an approved meta item
ergo3114 Aug 27, 2017
7383f1b
- Pull from upstream
ergo3114 Aug 28, 2017
b82a9b6
- Updated Test ConvertTo-HTML meta with invalid properties should thow
ergo3114 Aug 28, 2017
c0c4235
- added {get;set;} for the Transitional vaiable
ergo3114 Sep 1, 2017
19f2ea2
- Added warning message to meta tag from error
ergo3114 Sep 6, 2017
a380a35
- Pull warning from resource file
ergo3114 Sep 7, 2017
4970e36
- Removed commented out code
ergo3114 Sep 7, 2017
3a07e00
- Repush for Appveyor test
ergo3114 Sep 7, 2017
da80adc
- Repushing after rebase
ergo3114 Sep 7, 2017
a0ac1a1
- Changed to pull warning message from resouce file
ergo3114 Sep 8, 2017
605453f
- Changed test for invalid meta keys to look for key pair in verbose …
ergo3114 Sep 8, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
- added {get;set;} for the Transitional vaiable
  • Loading branch information
@ergo3114
ergo3114 committed Sep 1, 2017
commit c0c42350f8b63b22d6a405b1c26ffec13d8bfb51
15 changes: 13 additions & 2 deletions src/Microsoft.PowerShell.Commands.Utility/commands/utility/convert-HTML.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -276,7 +276,18 @@ public string Charset
/// <returns></returns>
[Parameter(ParameterSetName = "Page")]
[ValidateNotNullOrEmpty]
public SwitchParameter Transitional;
public SwitchParameter Transitional
{
get
{
return _transitional;
}
set
{
_transitional = true;
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can use an auto property for a switch parameter without getter/setter.

}
private bool _transitional = false;

/// <summary>
/// definitions for hash table keys
Expand Down Expand Up @@ -415,7 +426,7 @@ protected override void BeginProcessing()

if (!_fragment)
{
if(!Transitional)
if(!_transitional)
{
WriteObject("<!DOCTYPE html PUBLIC \"-//W3C//DTD XHTML 1.0 Strict//EN\" \"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd\">");
}
Expand Down
0