In the sync dialog, I get all the platforms and all the sites, but I came to click on the "Sync data", no matter what I have selected It always tell's me
"Are you sure you want to sync"

I have looked in the $form_state used in hosting_task_sync_form_validate, there always an empty ['values']['parameters']['source']

I could not figure why, even after messing a little with the code.

Comments

ybabel created an issue. See original summary.

ybabel’s picture

Ok now I have a clue : it's the "A custom Drush alias" code that interferes with those of the other sources.

function hosting_task_sync_form($node) {
...
  if (1) {
    $form['source']['custom_alias'] = array(
      '#type' => 'fieldset',
...

wether its drush alias that works, wether it's platforms sites, but not both of them

ybabel’s picture

Proposed patch, changing the order make it works ...

ybabel’s picture

Assigned: Unassigned » ybabel
Status: Active » Needs review
helmo’s picture

I don't like having the custom source as the first option.

We'll have to find a way to get the value of $form['source']['custom_alias']['text'] into another place.

helmo’s picture

Assigned: ybabel » Unassigned
Status: Needs review » Needs work
szeidler’s picture

I also ran into that issue.

For future reference: You can workaround the issue, using the drush alias you initially wanted to select in the custom alias field, till there will be any progress here.

Grimreaper’s picture

Version: 7.x-3.3 » 7.x-3.x-dev
Status: Needs work » Needs review
FileSize
919 bytes

Hello,

Here is a patch that fix the issue.

The problem is that the value of the hidden textfield is always send in the POST request when submitting the form. So it is an empty value.

A disabled textfield is not sent in the POST values, so disabling the textfield with the states API solved the problem.

Thanks for the review.

  • helmo committed c8d5968 on 7.x-3.x authored by Grimreaper
    Issue #2674264 by ybabel, Grimreaper: Sync data : No sources are...
helmo’s picture

Status: Needs review » Fixed

Thanks, that solved it. Committed.

Grimreaper’s picture

Thanks for the commit.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.