Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Oct 2011 at 12:42 UTC
Updated:
29 Jul 2014 at 20:03 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
avpadernoThis is not what I meant with "very shortly." I had to take care of family things. :-)
Comment #2
jhodgdonThanks for the spelling correction.
I'm not sure I agree with inserting "the" there though. I would say things like "Use function abc() to do x" not "Use the function abc() to do x". Isn't method pretty much the same word as function, but in a class? So I would say "Use method abc() to do x" also... and in this case, "Add the main keywords to the query by using method search_index()" seems fine to me without "the".
Comment #3
avpadernoThis patch only fixes the typo.
I added the article for consistency with the other sentence, "The functions search_expression_extract() and search_expression_insert() may also be helpful for adding custom search parameters to the search expression." I guess that using the article depends on how much definite is the method being mentioned. Is the method defined by a class that implements an interface, or is it the method defined by a specific class?
To me, "use function abc()" sounds like it is missing the article, and it sounds like saying "use coffee" instead of "use the coffee you find on the counter"; both are valid, but the latter is referring to specific coffee (i.e., it's not the coffee that you find in another place).
This is secondary, though. We could also argue that the other sentence can be simply written "search_expression_extract() and search_expression_insert() may also be helpful for adding custom search parameters to the search expression," as it should be implicit that
search_expression_extract()andsearch_expression_insert()are functions, and not methods. (It is implicit by the name, and for the fact I don't write "the methods" before them.)Comment #4
jhodgdonThis patch is fine, thanks!
Comment #5
dries commentedCommitted to 7.x and 8.x. Thanks!