feat: nested property for the where filter, #178

src/builtin/filters/array.ts

@@ -1,5 +1,6 @@
 import { isArray, last } from '../../util/underscore'
 import { isTruthy } from '../../render/boolean'
+import { FilterImpl } from '../../template/filter/filter-impl'
 
 export default {
   'join': (v: any[], arg: string) => v.join(arg === undefined ? ' ' : arg),
@@ -28,8 +29,11 @@ function slice<T> (v: T[], begin: number, length = 1): T[] {
   return v.slice(begin, begin + length)
 }
 
-function where<T> (arr: T[], property: string, value?: any): T[] {
-  return arr.filter(obj => value === undefined ? isTruthy(obj[property]) : obj[property] === value)
+function where<T extends object> (this: FilterImpl, arr: T[], property: string, expected?: any): T[] {
+  return arr.filter(obj => {
+    const value = this.context.getFromScope(obj, property)
+    return expected === undefined ? isTruthy(value) : value === expected
+  })
 }
 
 function uniq<T> (arr: T[]): T[] {

src/context/context.ts

@@ -1,9 +1,9 @@
-import * as _ from '../util/underscore'
 import { Drop } from '../drop/drop'
 import { __assign } from 'tslib'
 import { assert } from '../util/assert'
 import { NormalizedFullOptions, applyDefault } from '../liquid-options'
 import { Scope } from './scope'
+import { isArray, isNil, isString, isFunction, toLiquid } from '../util/underscore'
 
 export class Context {
   private scopes: Scope[] = [{}]
@@ -28,14 +28,18 @@ export class Context {
   }
   public get (path: string) {
     const paths = this.parseProp(path)
-    let ctx = this.findScope(paths[0]) || this.environments
-    for (const path of paths) {
-      ctx = readProperty(ctx, path)
-      if (_.isNil(ctx) && this.opts.strictVariables) {
+    const scope = this.findScope(paths[0]) || this.environments
+    return this.getFromScope(scope, paths)
+  }
+  public getFromScope (scope: object, paths: string[] | string) {
+    if (!isArray(paths)) paths = this.parseProp(paths)
+    return paths.reduce((scope, path) => {
+      scope = readProperty(scope, path)
+      if (isNil(scope) && this.opts.strictVariables) {
         throw new TypeError(`undefined variable: ${path}`)
       }
-    }
-    return ctx
+      return scope
+    }, scope)
   }
   public push (ctx: object) {
     return this.scopes.push(ctx)
@@ -115,11 +119,11 @@ export class Context {
   }
 }
 
-function readProperty (obj: Scope, key: string) {
-  if (_.isNil(obj)) return obj
-  obj = _.toLiquid(obj)
+export function readProperty (obj: Scope, key: string) {
+  if (isNil(obj)) return obj
+  obj = toLiquid(obj)
   if (obj instanceof Drop) {
-    if (_.isFunction(obj[key])) return obj[key]()
+    if (isFunction(obj[key])) return obj[key]()
     if (obj.hasOwnProperty(key)) return obj[key]
     return obj.liquidMethodMissing(key)
   }
@@ -130,17 +134,17 @@ function readProperty (obj: Scope, key: string) {
 }
 
 function readFirst (obj: Scope) {
-  if (_.isArray(obj)) return obj[0]
+  if (isArray(obj)) return obj[0]
   return obj['first']
 }
 
 function readLast (obj: Scope) {
-  if (_.isArray(obj)) return obj[obj.length - 1]
+  if (isArray(obj)) return obj[obj.length - 1]
   return obj['last']
 }
 
 function readSize (obj: Scope) {
-  if (_.isArray(obj) || _.isString(obj)) return obj.length
+  if (isArray(obj) || isString(obj)) return obj.length
   return obj['size']
 }
 

src/template/filter/filter.ts

@@ -8,9 +8,9 @@ type FilterArg = string|KeyValuePair
 export type FilterArgs = FilterArg[]
 
 export class Filter {
-  private name: string
+  public name: string
+  public args: FilterArgs
   private impl: FilterImplOptions
-  private args: FilterArgs
   private static impls: {[key: string]: FilterImplOptions} = {}
 
   public constructor (name: string, args: FilterArgs, strictFilters: boolean) {

src/template/value.ts

@@ -3,9 +3,9 @@ import { FilterArgs, Filter } from './filter/filter'
 import { Context } from '../context/context'
 
 export class Value {
+  public readonly filters: Filter[] = []
+  public readonly initial: string
   private strictFilters: boolean
-  private initial: string
-  private filters: Filter[] = []
 
   /**
    * @param str value string, like: "i have a dream | truncate: 3