Skip to content

Commit a2a6d7a

Browse files
committed
fix (searchparams): getName should be get
`getName` does not exist in `URLSearchParams`'s API, and the definition here was throwing an error because of this. It should actually be `get`. I checked all node versions prior to verify that it has indeed always been `get`
1 parent 5837f74 commit a2a6d7a

File tree

4 files changed

+23
-13
lines changed

4 files changed

+23
-13
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ Breaking changes:
99
New features:
1010

1111
Bugfixes:
12+
- `URLSearchParams.getName` does not exist, renamed to the correct name `get`
1213

1314
Other improvements:
1415

src/Node/URL/URLSearchParams.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ export const fromStringImpl = (str) => new url.URLSearchParams(str);
66
export const fromObjectImpl = (obj) => new url.URLSearchParams(obj);
77
export const appendParamImpl = (name, value, params) => params.append(name, value);
88
export const deleteImpl = (name, params) => params.delete(name);
9-
export const getNameImpl = (name, params) => params.getName(name);
9+
export const getImpl = (name, params) => params.get(name);
1010
export const getAllImpl = (name, params) => params.getAll(name);
1111
export const hasImpl = (name, params) => params.has(name);
1212
export const setImpl = (name, value, params) => params.set(name, value);

src/Node/URL/URLSearchParams.purs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ module Node.URL.URLSearchParams
55
, fromObject
66
, appendParam
77
, delete
8-
, getName
8+
, get
99
, getAll
1010
, has
1111
, set
@@ -57,10 +57,10 @@ delete name params = runEffectFn2 deleteImpl name params
5757

5858
foreign import deleteImpl :: EffectFn2 (String) (URLSearchParams) (Unit)
5959

60-
getName :: String -> URLSearchParams -> Effect (Maybe String)
61-
getName name params = map toMaybe $ runEffectFn2 getNameImpl name params
60+
get :: String -> URLSearchParams -> Effect (Maybe String)
61+
get name params = map toMaybe $ runEffectFn2 getImpl name params
6262

63-
foreign import getNameImpl :: EffectFn2 (String) (URLSearchParams) (Nullable String)
63+
foreign import getImpl :: EffectFn2 (String) (URLSearchParams) (Nullable String)
6464

6565
getAll :: String -> URLSearchParams -> Effect (Array String)
6666
getAll name params = runEffectFn2 getAllImpl name params

test/Main.purs

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,18 @@ module Test.Main where
22

33
import Prelude
44

5-
import Effect (Effect)
6-
import Node.URL as URL
7-
import Node.URL.URLSearchParams as URL.Search
8-
import Foreign (unsafeReadTagged)
9-
import Data.Bifunctor (lmap)
10-
import Control.Monad.Except (runExcept)
115
import Control.Monad.Error.Class (liftEither)
12-
import Effect.Exception (error)
6+
import Control.Monad.Except (runExcept)
7+
import Data.Bifunctor (lmap)
8+
import Data.Map as Map
9+
import Data.Maybe (Maybe(..))
1310
import Data.Traversable (for)
1411
import Data.Tuple (Tuple(..))
15-
import Data.Map as Map
12+
import Effect (Effect)
13+
import Effect.Exception (error)
14+
import Foreign (unsafeReadTagged)
15+
import Node.URL as URL
16+
import Node.URL.URLSearchParams as URL.Search
1617
import Test.Assert (assertEqual)
1718

1819
main Effect Unit
@@ -54,3 +55,11 @@ main = do
5455
, Tuple "v" [ "a", "b" ]
5556
]
5657
}
58+
59+
v <- URL.Search.get "v" search
60+
61+
assertEqual
62+
{ actual: v
63+
, expected: Just "a"
64+
}
65+

0 commit comments

Comments
 (0)