Skip to content

Documentation fixes #1379

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

Merged
merged 5 commits into from
Oct 14, 2017
Merged

Documentation fixes #1379

merged 5 commits into from
Oct 14, 2017

Conversation

fredericDelaporte
Copy link
Member

Fixes #788, #823, #824, #1165 and #1179. (This time I have tested generating the doc locally before issuing the PR.)

@@ -78,6 +79,7 @@ public static string Version
public const string ConnectionStringName = "connection.connection_string_name";

// Unused, Java-specific
// But has many code usage though.
public const string SessionFactoryName = "session_factory_name";
Copy link
Member Author

Choose a reason for hiding this comment

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

Not switched to obsolete because its many code usages imply a lot of changes, and so I am not sure it has really not any use. (And I do not intend to investigate further the subject.)

// support the version taking an array of the names of the columns to
// be returned (via its RETURNING clause). No other driver seems to
// support this overloaded version.
DefaultProperties[Environment.UseGetGeneratedKeys] = "false";
Copy link
Member Author

Choose a reason for hiding this comment

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

This thing was set but never read. Its corresponding feature is not implemented.

@hazzik hazzik added this to the 5.0.1 milestone Oct 14, 2017
@hazzik hazzik merged commit 903494f into nhibernate:master Oct 14, 2017
@fredericDelaporte fredericDelaporte deleted the DocFixes branch October 15, 2017 09:02
@hazzik hazzik changed the title Doc fixes Documentation fixes Oct 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants