Just a suggestion but I've accidentally clicked the re-index button a couple of times now (Doh!).

Can a "Are you sure you want to re-index? confirmation page be added to keep those of us who click before we think from re-indexing when it's not necessary?

Thanks,
Kevin

CommentFileSizeAuthor
#3 confirm.patch3.69 KBrobertdouglass

Comments

pwolanin’s picture

Version: 6.x-1.0-rc1 » 6.x-1.x-dev

should there be a threshold? For a smaller site (e.g. 1000 nodes) this is a pretty minor action.

robertdouglass’s picture

I think adding a confirmation form is in line with normal Drupal usage.

robertdouglass’s picture

Status: Active » Needs review
StatusFileSize
new3.69 KB
robertdouglass’s picture

Peter - let's avoid the threshold. Too hard to tell when it's appropriate and when not, and it would add extra code for no good reason.

I'm overall neutral on the feature request, but I do think it is probably more in line with typical Drupal behavior, and there's request for the feature, thus I'm inclined to commit to both branches pending review.

pwolanin’s picture

@robert - it's ok with me

robertdouglass’s picture

Status: Needs review » Fixed

Committed to 6.1 and 6.2.

Status: Fixed » Closed (fixed)

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

aufumy’s picture

 function apachesolr_clear_index($form, &$form_state) {
+  return apachesolr_comfirm_clear_index($form, $form_state);
+}

should maybe be

 function apachesolr_clear_index($form, &$form_state) {
+  return apachesolr_clear_index_confirm($form, $form_state);
+}