add test, handle Vector{Any}

Author: BeastyBlacksmith

src/app/dashapp.jl

@@ -48,7 +48,7 @@ function Base.getindex(component::DashBase.Component, id::AbstractString)
   component.id == id && return component
   hasproperty(component, :children) || return nothing
   cc = component.children
-  cc isa Union{VecChildTypes, DashBase.Component} ? cc[id] : nothing
+  cc isa Union{VecChildTypes, DashBase.Component, Vector{Any}} ? cc[id] : nothing
 end
 function Base.getindex(children::VecChildTypes, id::AbstractString)
   for element in children
@@ -57,6 +57,14 @@ function Base.getindex(children::VecChildTypes, id::AbstractString)
     el !== nothing && return el
   end
 end
+function Base.getindex(children::AbstractVector, id::AbstractString)
+  for element in children
+    hasproperty(element, :id) || continue
+    element.id == id && return element
+    el = element[id]
+    el !== nothing && return el
+  end
+end
 
 #only name, index_string and layout are available to set
 function Base.setproperty!(app::DashApp, property::Symbol, value)

test/core.jl

@@ -200,3 +200,17 @@ end
     end
     @test_throws ErrorException make_handler(app)
 end
+
+@testset "Index by id" begin
+    app = dash()
+    app.layout = html_div() do
+            dcc_input(id = "my-id", value="initial value", type = "text"),
+            html_div(id = "my-div", children = [
+                html_div(),
+                "string",
+                html_div(id = "target")
+            ]),
+            html_div(id = "my-div2")
+        end
+    @test app["target"].id == "target"
+end