Changes between Initial Version and Version 1 of Ticket #31045, comment 24
- Timestamp:
- 01/31/2015 07:26:35 PM (10 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #31045, comment 24
initial v1 1 After talking to @boonebgorges in Slack, my best suggestion is that we go with `clause_id` for the argument name. I share his concerns about using the word "alias" in the argument name because of the implication that it any effect on the actual alias used when building the query.1 After talking to @boonebgorges in Slack, my best suggestion is that we go with `clause_id` for the argument name. I share his concerns about using the word "alias" in the argument name because of the implication that it has any effect on the actual alias used when building the query. 2 2 3 The purpose of the discussion is to as simply as possible, draw a line between setting the value in the clause and referencing it in the orderby definition. By avoiding the use of the word "orderby" in the new argument name, we also don't lock ourselves into that vernacular should we find another good use for this argument in the future.3 The purpose of the discussion is to -- as simply as possible -- draw a line between setting the value in the clause and referencing it in the orderby. By avoiding the use of the word "orderby" in the new argument name, we also don't lock ourselves into that vernacular should we find another good use for this argument in the future.