Just so I can track it for http://drupal.org/project/clientside_validation

CommentFileSizeAuthor
#5 1309504-multivalue-settings-integration.patch8.52 KBjelle_s
#4 clientside_validation_multivalue_settings.zip1.54 KBjelle_s
#4 1309504-multivalue-settings-integration.patch8.29 KBjelle_s

Comments

goron’s picture

goron commented

Hi attiks,

Is this integration something that needs to happen in Clientside Validation, or Multivalue Settings? There's some enhanced validation that this module adds to forms. Is that simple to integrate into Clientside Validation?

Thanks.

attiks’s picture

attiks commented

@goron: the way clientside validation works is by looking at the fapi elements, so if your module can expose this information somehow, we can detect it. The integration itself can happen inside either module, we added hooks to clientside validation to support this.

attiks’s picture

attiks commented

Integration is working, but needs a core patch #208611-49: Add drupal_array_merge_deep() and drupal_array_merge_deep_array() to stop drupal_add_js() from adding settings twice

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
StatusFileSize
new 1309504-multivalue-settings-integration.patch8.29 KB
new clientside_validation_multivalue_settings.zip1.54 KB

In attachment the patch to apply to clientside validation and a .zip of the submodule

this should work in combination with the patch in the previous post

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
StatusFileSize
new 1309504-multivalue-settings-integration.patch8.52 KB

sorry, there was an error left in the previous patch.

Correct patch: