Skip to content

Commit f534739

Browse files
committed
Simple cleanup to improve interop (data first)
1 parent 6a622fb commit f534739

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

src/Elmish.WPF.Tests/DynamicViewModelTests.fs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ module Helpers =
129129
AutoRequery = autoRequery }
130130
|> CmdData
131131
|> BaseBindingData
132-
|> createBinding) name
132+
|> Binding.createBinding) name
133133

134134

135135
let internal subModel

src/Elmish.WPF/Binding.fs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,13 @@ open Elmish
88
module Binding =
99
open BindingData
1010

11+
let internal createBinding data name =
12+
{ Data = data
13+
Name = name }
14+
1115
let internal mapData f binding =
12-
{ Name = binding.Name
13-
Data = binding.Data |> f }
16+
{ Data = binding.Data |> f
17+
Name = binding.Name }
1418

1519
/// Maps the model of a binding via a contravariant mapping.
1620
let mapModel (f: 'a -> 'b) (binding: Binding<'b, 'msg>) = f |> mapModel |> mapData <| binding

src/Elmish.WPF/DynamicViewModel.fs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,6 @@ type Binding<'model, 'msg> =
1818
[<AutoOpen>]
1919
module internal Helpers =
2020

21-
let createBinding data name =
22-
{ Name = name
23-
Data = data }
24-
2521
type SubModelSelectedItemLast with
2622
member this.CompareBindings() : Binding<'model, 'msg> -> Binding<'model, 'msg> -> int =
2723
fun a b -> this.Recursive(a.Data) - this.Recursive(b.Data)

0 commit comments

Comments
 (0)