tablacus / TablacusExplorerAddons

Add-ons for Tablacus Explorer
http://tablacus.github.io/TablacusExplorerAddons/
MIT License
87 stars 12 forks source link

Sync select - "Extension" option should be ignored for folders #353

Closed ccfs closed 1 year ago

ccfs commented 1 year ago

Folders do not have "extension", but the add-on cannot handle this correctly. @bblackmoor and @tablacus, please help to solve this problem. Thanks.

ss fl

bblackmoor commented 1 year ago

I tested this in my sandbox. My tests show:

1) When "File extension" (NOTE: This is not a clear label in English. Inverting the logic and labeling it "Ignore extension" would be clearer, and more intuitive.) is selected (the default):

image

"TablacusExplorer" and "TablacusExplorerAddons" are selected. "TablacusExplorer.old" is not. This is as expected.

2) When "File extension" (see note above) is not selected:

image

"TablacusExplorer", "TablacusExplorer.old", and "TablacusExplorerAddons" are selected. This is as expected.

The setting appears to be functioning as expected.

If I appear to have misunderstood something, please let me know.

ccfs commented 1 year ago

"File extension" is the string after the last dot of the name of a FILE. It indicates a file type and the associated application. However FOLDER is a type itself. Anything in the name, including any dot, is the name.

Therefore in my example above, it is correct for "file.v2" but incorrect for "folder.v2".

bblackmoor commented 1 year ago

A directory is a unique type of file that contains only the information needed to access files or other directories. The extension is just part of the name. The add-on appears to be functioning as expected.

bblackmoor commented 1 year ago

Besides, what would be the alternative? That the extension is always ignored for directories? Even if the extension on a file was somehow different from the extension on a directory (it's not), is that a reason to remove the choice whether to ignore the extension on directories?

ccfs commented 1 year ago

What I stated is the Microsoft's definition of "file extension". Of course some people may have a different understanding. Whatever. Users can try ticking or clearing the option to fit their needs.

So we can keep the add-on unchanged. Thanks.