addresses #322 by adding get filter

tests, code, docs, changes updated

Signed-off-by: Sean Corfield <[email protected]>

Author: seancorfield

README.md

@@ -65,6 +65,7 @@ A fast, [Django](https://docs.djangoproject.com/en/dev/ref/templates/builtins/)
 [take](#take)
 [drop](#drop)
 [drop-last](#drop-last)
+[get](#get)
 [get-digit](#get-digit)
 [hash](#hash)
 [join](#join)
@@ -513,6 +514,8 @@ Similar to drop:
 
 `(render "{{seq-of-some-sort|drop-last:4}}" {:seq-of-some-sort [:dog :cat :bird :bird :bird :is :the :word]})` =\> `[:dog :cat :bird :bird]`
 
+#### get
+`(render "{{my-map|get:@key}}" {:my-map {:a 1 :b 2} :key :b})` => `2`
 
 #### get-digit
 `(render "{{tis-a-number|get-digit:1}}" {:tis-a-number 12.34567})` => `7`

changes.md

@@ -1,3 +1,4 @@
+* 1.12.next - [add `get` filter](https://github.com/yogthos/Selmer/issue/322)
 * 1.12.69 - fix relfection warning
 * 1.12.68 - fix handling nulls when parsing script tags
 * 1.12.67 - fix reflection warning parsing uri

src/selmer/filters.clj

@@ -57,7 +57,7 @@ map. The rest of the arguments are optional and are always strings."
         (-> (.getTime ^java.sql.Time d)
             (Instant/ofEpochMilli)
             (LocalDateTime/ofInstant (ZoneId/systemDefault)))
-        
+
         (instance? java.sql.Timestamp d)
         (-> (.getTime ^java.sql.Timestamp d)
             (Instant/ofEpochMilli)
@@ -70,7 +70,7 @@ map. The rest of the arguments are optional and are always strings."
         (-> (.toInstant ^java.util.Date d)
             (.atZone (ZoneId/systemDefault))
             (.toLocalDateTime))
-        
+
         (instance? java.time.Instant d)
         (-> (.atZone ^java.time.Instant d (ZoneId/systemDefault))
             (.toLocalDateTime))
@@ -553,7 +553,10 @@ map. The rest of the arguments are optional and are always strings."
                   (throw (Exception. (str number " does not appear to be a valid phone number"))))))
 
             :name
-            name}))
+            name
+
+            :get
+            get}))
 
 
 (defn get-filter

test/selmer/core_test.clj

@@ -1130,6 +1130,21 @@
   (is (= "quux"                                             ;; test nested lookup
          (render "{{name|default:@foo.bar.baz}}" {:name nil :foo {:bar {:baz "quux"}}}))))
 
+(deftest filter-get-tests
+  (is (= "bar"
+         (render "{{data|get:\"foo\"}}" {:data {"foo" "bar" "baz" "quux"}})))
+  (is (= "quux"
+         (render "{{data|get:\"baz\"}}" {:data {"foo" "bar" "baz" "quux"}})))
+  (is (= ""
+         (render "{{data|get:\"nonexistent\"}}" {:data {"foo" "bar" "baz" "quux"}})))
+  (is (= "default-value"
+         (render "{{data|get:\"nonexistent\":\"default-value\"}}" {:data {"foo" "bar" "baz" "quux"}})))
+  (is (= "2"
+         (render "{{my-map|get:@key}}" {:my-map {:a 1 :b 2} :key :b})))
+  (let [data {:x {"a" 1 "b" 2} :ys ["a" "b"]}]
+    (is (= "12"
+           (render "{% for y in ys %}{{x|get:@y}}{% endfor %}" data)))))
+
 (deftest custom-resource-path-setting
   (is (nil? *custom-resource-path*))
   (do