-
Notifications
You must be signed in to change notification settings - Fork 34
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.
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
Problem with schemas that include a period "." #27
Comments
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! |
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. |
Beta works great! Thanks for the quick fix! |
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. |
See release v1.4-beta2. |
Once again, works well and thanks for the quick fix! |
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"
The text was updated successfully, but these errors were encountered: