SSMS-Schema-Folders icon indicating copy to clipboard operation
SSMS-Schema-Folders copied to clipboard

Problem with schemas that include a period "."

Open cawise opened this issue 2 years ago • 7 comments

We are using multi-part schemas ("sub-schemas", if you like) that include a dot to separate the "parent schema" from the "sub-schema" like [Technology.Logging]. In most cases, this extension works as expected. However, when the table name happens to be a prefix/substring of the "sub-schema", it groups weirdly -- as though the table name is removed from the schema name and leaves any remaining letters.

In the image below, you can see a few examples of this. [Technology.LandingPages].[LandingPage] yields "Technologys" [Technology.Logging].[Log] yields "Technologyging"

image001

cawise avatar Jul 27 '22 15:07 cawise

Should we expect a release with this fix in soon or are you waiting for more issues / features to add before doing one? Just asking to determine if we should pull the code down and do our own internal release or wait for an official one.

Thanks for the quick fix!

scottkuhl avatar Jul 29 '22 14:07 scottkuhl

I will be adding Folder Levels for the next release and need to update the localisation of the new features.

Since I'm not sure when it will be complete, I have done a beta release of the current build.

nicholas-ross avatar Jul 30 '22 03:07 nicholas-ross

Beta works great! Thanks for the quick fix!

cawise avatar Aug 01 '22 21:08 cawise

Here's a related issue, but I can open as a separate item if you like.

With these "multipart" schema names, when the Object Display Options > Rename Node option is set to True, it only strips off the first part of the schema name.

image

cawise avatar Aug 03 '22 19:08 cawise

Thank you. I am aware of it.

It was a performance consideration I made at the time. To get the accurate object name a call to GetService for each node is required. I'm already doing this once but the rename was a different method and didn't want to repeat the same call. After merging in my folder level code I have decided it needs to be rewritten to parse all the folder levels and name with one call.

nicholas-ross avatar Aug 04 '22 02:08 nicholas-ross

See release v1.4-beta2.

nicholas-ross avatar Aug 04 '22 07:08 nicholas-ross

Once again, works well and thanks for the quick fix!

cawise avatar Aug 08 '22 15:08 cawise