alphaleonis / alphafs Goto Github PK

Issue by Yomodo from Thursday Jan 08, 2015 at 13:54 GMT
Originally opened as #32

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/32/commits

Issue by alphaleonis from Sunday Jan 04, 2015 at 21:16 GMT
Originally opened as #9

There are some items that were not merged over correctly. Fix this.

Issue by Yomodo from Thursday Jan 08, 2015 at 23:04 GMT
Originally opened as #37

Issue by alphaleonis from Monday Jan 05, 2015 at 07:32 GMT
Originally opened as #13

A new parameter has been introduced on various methods; isFullPath. This is currently of type bool? which makes its use unclear without reading the documentation (i.e. what does null mean?).
I suggest we change this parameter to an enumeration so that its meaning is clear instead.
For example:

enum PathFormat 
{
   Auto,
   Standard,
   ExtendedLength
}

or something similar.

Issue by alphaleonis from Friday Jan 02, 2015 at 19:33 GMT
Originally opened as #3

Encryption/decryption happens automatically with most of the regular api calls if the right cert is installed.

However there are situations when if the drive was taken out of the computer for forensic analysis, you would need to access the encrypted files.

In that situation there is usually a delegated decryption user's certificate is stated in the encrypted files metadata, usually it is a active directory primary administrator who allowed the EFS to be used by regular users. So you would need to acquire the cert from that admin to the computer which will try to decrypt affected files.

I think think we could add some extra methods when the right Windows license with EFS lands on one of our hands.

Issue by Yomodo from Thursday Jan 08, 2015 at 11:30 GMT
Originally opened as #30

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/30/commits

Issue by alphaleonis from Tuesday Jan 06, 2015 at 19:49 GMT
Originally opened as #19

Issue by alphaleonis from Monday Jan 05, 2015 at 16:58 GMT
Originally opened as #15

In the process refactored out a bit of duplicated code that managed this parameter, and changed the signature of some (internal) methods from accepting a large number of bool parameters to accepting a single enum value which improves readability.

Issue #13

Care to review this @Yomodo ?

alphaleonis included the following code: https://github.com/alphaleonis/AlphaFS/pull/15/commits

Issue by alphaleonis from Monday Jan 05, 2015 at 07:24 GMT
Originally opened as #11

A merge seems to have removed the proper markdown formatting.

Issue by alphaleonis from Wednesday Jan 07, 2015 at 14:59 GMT
Originally opened as #27

Issue by alphaleonis from Friday Jan 02, 2015 at 20:23 GMT
Originally opened as #4

There are a ton of warnings about unresolved links, and in many cases the Remarks-section is nested inside the <summary> tags which creates bad output. Review this and try to fix these issues.

Issue by Yomodo from Friday Jan 09, 2015 at 11:07 GMT
Originally opened as #40

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/40/commits

Issue by alphaleonis from Wednesday Jan 07, 2015 at 12:24 GMT
Originally opened as #23

These methods were added for 1.6, but have somewhat confusing names, might be confused with adding a .NET Stream.
These methods exist in AlternateDataStreamInfo, DirectoryInfo, FileInfo, Directory and File.

Suggested renames for methods:

• AddStream => AddAlternateDataStream
• RemoveStream => RemoveAlternateDataStream
• EnumerateStreams => EnumerateAlternateDataStreams
• GetStreamSize => GetAlternateDataStreamSize

Issue by alphaleonis from Tuesday Jan 06, 2015 at 19:50 GMT
Originally opened as #20

Issue by Yomodo from Friday Jan 09, 2015 at 10:22 GMT
Originally opened as #39

Issue by alphaleonis from Friday Jan 02, 2015 at 19:33 GMT
Originally opened as #2

Issue by alphaleonis from Tuesday Jan 06, 2015 at 19:55 GMT
Originally opened as #21

Do this just before release for simplicity.

Issue by alphaleonis from Saturday Jan 03, 2015 at 20:27 GMT
Originally opened as #6

From 6182 warnings to 0 in ... well... way too much time. ;)

alphaleonis included the following code: https://github.com/alphaleonis/AlphaFS/pull/6/commits

Issue by Yomodo from Thursday Jan 08, 2015 at 16:16 GMT
Originally opened as #34

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/34/commits

Issue by alphaleonis from Monday Jan 05, 2015 at 18:04 GMT
Originally opened as #16

Version 1.6 will be released on NuGet.

Issue by Yomodo from Friday Jan 09, 2015 at 19:22 GMT
Originally opened as #49

-Added AsLongPath member to GetFullPathOptions enum;
-Removed all overloaded methods with asLongPath parameter;

Issue by Yomodo from Thursday Jan 08, 2015 at 17:59 GMT
Originally opened as #35

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/35/commits

Issue by alphaleonis from Friday Jan 02, 2015 at 19:32 GMT
Originally opened as #1

The System.IO.FileSystemWatcher class has not been replicated in this project which it should be.
If you want completeness.
Thanks!

Issue by hnorihiro from Friday Jan 09, 2015 at 11:16 GMT
Originally opened as #41

1.I format the G drive

AlphaFS(2015/1/8)

DirectoryInfo info = new DirectoryInfo(@"G:\");
info.GetFileSystemInfos();//Directory not found Exception

.NET

System.IO.DirectoryInfo info = new System.IO.DirectoryInfo(@"G:\");
info.GetFileSystemInfos();//Success!!

Can you help me?

Issue by alphaleonis from Wednesday Jan 07, 2015 at 12:48 GMT
Originally opened as #25

AddDirectorySeparator should be renamed to AddTrailingDirectorySeparator to clarify its purpose. Its documentation should be updated to reflect that this is what it is doing. Also documentation needs to be updated to reflect what the difference between this method and GetSuffixedDirectoryName is.

RemoveDirectorySeparator should consequently be renamed to RemoveTrailingDirectorySeparator. Same updates to documentation.

Issue by Yomodo from Friday Jan 09, 2015 at 18:02 GMT
Originally opened as #47

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/47/commits

Issue by Yomodo from Thursday Jan 08, 2015 at 00:34 GMT
Originally opened as #29

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/29/commits

Issue by Yomodo from Thursday Jan 08, 2015 at 12:10 GMT
Originally opened as #31

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/31/commits

Issue by alphaleonis from Friday Jan 02, 2015 at 20:30 GMT
Originally opened as #5

...nged layout from deprecated to VS2013.

• Moved documentation project and renamed it so it could be included in the solution.
• Updated license text in documentation to include correct year.

alphaleonis included the following code: https://github.com/alphaleonis/AlphaFS/pull/5/commits

Issue by Yomodo from Thursday Jan 08, 2015 at 23:27 GMT
Originally opened as #38

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/38/commits

Issue by alphaleonis from Wednesday Jan 07, 2015 at 10:09 GMT
Originally opened as #22

If a test cannot run due to environmental problems, such as the computer not being part of a domain, that test should be marked inconclusive. Today these tests fail.

They should instead contain something like the following in the beginning of the test method:

if (testCannotExecute)
   Assert.Inconclusive("This test cannot be run because XXX");

Issue by Yomodo from Monday Jan 05, 2015 at 02:46 GMT
Originally opened as #10

-Restored File.GetFileSystemEntryInfo();
-Add items to enum DirectoryEnumerationOptions;
-Created constructor for class FindFileSystemEntryInfo();

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/10/commits

Issue by Yomodo from Thursday Jan 08, 2015 at 22:28 GMT
Originally opened as #36

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/36/commits

Issue by alphaleonis from Monday Jan 05, 2015 at 20:10 GMT
Originally opened as #17

Fixes #14

Note! Merge and review this after merging #13

@Yomodo Care to review this?

alphaleonis included the following code: https://github.com/alphaleonis/AlphaFS/pull/17/commits

Issue by Yomodo from Thursday Jan 08, 2015 at 14:08 GMT
Originally opened as #33

Windows Portable Device (WPD) seems a worthy addition to AlphaFS.

There is already working code that needs to be expanded and improved upon.

Issue by Yomodo from Friday Jan 09, 2015 at 19:24 GMT
Originally opened as #50

Path.GetLongPath("C:") returns C:\ instead of \?\C:\

I guess it should?

Issue by alphaleonis from Wednesday Jan 07, 2015 at 17:24 GMT
Originally opened as #28

Currently they accept only a string-array, which is unnecessarily restrictive. A better option would be to have the "AddStream" method return a Stream, and let the user write the data as he/she pleases. It is a "stream" after all.

Issue by Yomodo from Friday Jan 09, 2015 at 19:21 GMT
Originally opened as #48

-Update remarks for Directory.GetCurrentDirectory/SetCurrentDirectory()/GetFullPath() methods;
-Improved upon Path- and Directory class unit tests;

Issue by alphaleonis from Sunday Jan 04, 2015 at 10:42 GMT
Originally opened as #7

alphaleonis included the following code: https://github.com/alphaleonis/AlphaFS/pull/7/commits

Issue by alphaleonis from Sunday Jan 04, 2015 at 11:10 GMT
Originally opened as #8

alphaleonis included the following code: https://github.com/alphaleonis/AlphaFS/pull/8/commits

Issue by alphaleonis from Monday Jan 05, 2015 at 10:48 GMT
Originally opened as #14

The File and Directory classes are huge by now and littered with regions. Split these classes into multiple files so that they become more manageable and with less regions.

Issue by alphaleonis from Tuesday Jan 06, 2015 at 08:58 GMT
Originally opened as #18

Issue by alphaleonis from Wednesday Jan 07, 2015 at 12:27 GMT
Originally opened as #24

This makes it more clear what the method does. Is abit ambiguous now.

Issue by Yomodo from Friday Jan 09, 2015 at 17:56 GMT
Originally opened as #46

For consistency with: PathFormat.FulPath and PathFormat.LongFullPath

Issue by Yomodo from Friday Jan 09, 2015 at 17:31 GMT
Originally opened as #44

Issue by Yomodo from Friday Jan 09, 2015 at 17:10 GMT
Originally opened as #43

-Added overloads with PathFormat argument to class AlternateDataStreamInfo(), BackupFileStream() and Directory.Copy();
-Updated comments and rearranged methods.

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/43/commits

Issue by alphaleonis from Monday Jan 05, 2015 at 07:25 GMT
Originally opened as #12

Fixes #11

alphaleonis included the following code: https://github.com/alphaleonis/AlphaFS/pull/12/commits

Issue by Yomodo from Friday Jan 09, 2015 at 15:06 GMT
Originally opened as #42

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/42/commits

Issue by alphaleonis from Wednesday Jan 07, 2015 at 14:58 GMT
Originally opened as #26

Issue by Yomodo from Friday Jan 09, 2015 at 17:33 GMT
Originally opened as #45

Yomodo included the following code: https://github.com/alphaleonis/AlphaFS/pull/45/commits