DOCS SQLSelect::value() should be execute()

[christopherdarling]

No description provided.

[dhensby]

Interesting, it looks like it should really be ->execute()->value() but this also means this bit of code is wrong 😬 https://github.com/silverstripe/silverstripe-framework/blob/4.10.4/src/ORM/Queries/SQLSelect.php#L587-L593

[christopherdarling]

https://github.com/silverstripe/silverstripe-framework/blob/4/src/ORM/Queries/SQLSelect.php#L587-L593 looks alright to me line 593 is calling value on a Query instance

silverstripe-framework/src/ORM/Connect/Query.php

Line 123 in dcdc255

public function value()

[GuySartorelli]

Suggested change

	$count = $query->setFrom('Member')->setSelect('COUNT(*)')->execute();
	$count = $query->setFrom('Member')->setSelect('COUNT(*)')->execute()->value();

SQLSelect::execute() returns a Query instance, from which you then get the value: https://api.silverstripe.org/4/SilverStripe/ORM/Queries/SQLSelect.html#method_execute

[GuySartorelli]

Note: This should not be merged here. I have started the process of migrating docs to a new repository so this PR should be closed and a new one should be created there.

[GuySartorelli]

Closing due to inaction - feel free to reopen this in the developer-docs repo if it's bothering you.