Skip to content
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

fix(docs): correct age filtering description #1454

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

saidmtanzania
Copy link
Contributor

@saidmtanzania saidmtanzania commented Sep 9, 2024

Issue
The documentation contains an error in the description of filtering people by age. The current text incorrectly states that the filter is for people "under 18 and over 90," which contradicts the actual query logic.

Fix
Correct the description to accurately reflect the query logic, which filters out people "under 90 and over 18."

Changes Made
Updated the description to correctly state the age filtering criteria.

Before

Say you want to filter out people under 18 and over 90, but you don't want to hit the database again:

<cfquery name="filteredQuery" dbtype="query">
  SELECT     Name, Age, Location
  FROM    myQuery
  WHERE    Age >= 18
            AND Age <= 90
</cfquery>

filteredQry contains the desired records.

After

Say you want to filter out people under 90 and over 18, but you don't want to hit the database again:

<cfquery name="filteredQuery" dbtype="query">
  SELECT     Name, Age, Location
  FROM    myQuery
  WHERE    Age >= 18
            AND Age <= 90
</cfquery>

filteredQry contains the desired records.

Updated the documentation to accurately describe the age filtering criteria. The previous text incorrectly stated filtering for people under 18 and over 90, which contradicted the query logic. The corrected description now states filtering for people under 90 and over 18.
@CLAassistant
Copy link

CLAassistant commented Sep 9, 2024

CLA assistant check
All committers have signed the CLA.

@zspitzer zspitzer merged commit 79c1532 into lucee:master Sep 17, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants