Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Fixed: Form/Element/Collection can't remove all elements #7447

Closed
wants to merge 3 commits into from
Closed

Fixed: Form/Element/Collection can't remove all elements #7447

wants to merge 3 commits into from

Commits on Apr 14, 2015

  1. Fixed testCanRemoveAllElementsIfAllowRemoveIsTrue test 

    this was checking empty collection that's why it had 0 after `$collection->populateValues(array());`
    @svycka
    svycka committed Apr 14, 2015
    Configuration menu
    Copy the full SHA
    ecf645f View commit details
    Browse the repository at this point in the history

  2. should remove all elements if data is empty array

    @svycka
    svycka committed Apr 14, 2015
    Configuration menu
    Copy the full SHA
    a4eb7b0 View commit details
    Browse the repository at this point in the history

  3. changed bind and setData order 

    not sure I am allowed to do this
    @svycka
    svycka committed Apr 14, 2015
    Configuration menu
    Copy the full SHA
    a6fc468 View commit details
    Browse the repository at this point in the history