CiviCRM Community Forums (archive)

CiviCRM Forms have URLs like 'civicrm/contribute/transact?reset=1&id=1'. If you want to make this accessible by the name 'donate' you'll soon discover that Drupal's URL Aliases module doesn't support the arguments (indicated by the '?' and '&' characters). This HOWTO should help. The downside is you'll need to add a module and hack a Drupal core file (sorry).

Install the URL Alter module and add the following code for custom_url_rewrite_inbound based on instructions at http://drupal.org/node/118072#comment-2857522 (and extending that code to match both '?' and '&' characters)...

/**
 * Allows query strings in URL aliases
 *  : http://drupal.org/node/118072#comment-2857522
 */
  $parts = preg_split('/[\?\&]/', $result);

  if (count($parts) > 1) {
    $result = array_shift($parts);

    foreach ($parts as $part) {
      list($key, $value) = explode('=', $part);
      $_GET[$key] = $value;
    }
  }

A side-effect of this change is that when creating menu items which refer to CiviCRM pages, the arguments get stripped (eg, the event id on a registration page). (When you save a menu item, Drupal normalises the path, which strips the arguments.)

Make the following change to modules/menu/menu.admin.inc to prevent the query arguments from being stripped from menu items.

/**
   * Validate form values for a menu link being added or edited.
   */
  function menu_edit_item_validate($form, &$form_state) {
    $item = &$form_state['values']['menu'];
 -  $normal_path = drupal_get_normal_path($item['link_path']);
 +  /*
 +   * drupal_get_normal_path() calls custom_url_rewrite_inbound()
 +   * which will remove any query arguments from the URL.  So
 +   * replace  the call  to  drupal_get_normal_path()  with the
 +   * code from that function, sans custom_url_rewrite_inbound().
 +   *
 +   * $normal_path = drupal_get_normal_path($item['link_path']);
 +   */
 +  $normal_path = $item['link_path'];
 +  if ($src = drupal_lookup_path('source', $item['link_path'], '')) {
 +    $normal_path = $src;
 +  } 
    if ($item['link_path'] != $normal_path) {

As this is a hack of core Drupal, you'll need to fix this every time you upgrade to a new version.

Thanks much for this reply. I agree with Xavier, this would be a nice core feature. As I know folks have enjoyed similar functionality in Drupal.

Hi,

Will look at what kurund did for the export, see if I can see how I can apply it to add a hook on crmUrl functions... and/or IRC.

acbs, could you code the other part (using the hook to alter the url + handle the new url to call the right civicrm functions) ? (or financial contribution to make it happen) ?

 
X+

sorry not sure I follow you. what do you mean by inbound?

X+

WHAT is with the [code link]? I get the message:
"Sorry, you are not allowed to post external links."
whenever I try to wrap code... grrrr. FWIW, that captcha is the most annoying! Can anyone read those characters?

[end rant]

Anyway....

I get this error message when I install the url_alter and apply fix in message 1.
Parse error: syntax error, unexpected T_VARIABLE in /home/bcaamno/public_html/modules/menu/menu.admin.inc on line 346

Line 246 is:    $normal_path = $item['link_path'];

My full modified menu_edit_item_validate function is:

function menu_edit_item_validate($form, &$form_state) {
  $item = &$form_state['values']['menu'];
  //$normal_path = drupal_get_normal_path($item['link_path']);
  /*
    * drupal_get_normal_path() calls custom_url_rewrite_inbound()
    * which will remove any query arguments from the URL.  So
    * replace  the call  to  drupal_get_normal_path()  with the
    * code from that function, sans custom_url_rewrite_inbound().
    *
    * $normal_path = drupal_get_normal_path($item['link_path']);
    */
   $normal_path = $item['link_path'];
   if ($src = drupal_lookup_path('source', $item['link_path'], '')) {
     $normal_path = $src;
   }
  if ($item['link_path'] != $normal_path) {
    drupal_set_message(t('The menu system stores system paths only, but will use the URL alias for display. %link_path has been stored as %normal_path', array('%link_path' => $item['link_path'], '%normal_path' => $normal_path)));
    $item['link_path'] = $normal_path;
  }
  if (!menu_path_is_external($item['link_path'])) {
    $parsed_link = parse_url($item['link_path']);
    if (isset($parsed_link['query'])) {
      $item['options']['query'] = $parsed_link['query'];
    }
    else {
      // Use unset() rather than setting to empty string
      // to avoid redundant serialized data being stored.
      unset($item['options']['query']);
    }
    if (isset($parsed_link['fragment'])) {
      $item['options']['fragment'] = $parsed_link['fragment'];
    }
    else {
      unset($item['options']['fragment']);
    }
    if ($item['link_path'] != $parsed_link['path']) {
      $item['link_path'] = $parsed_link['path'];
    }
  }
  if (!trim($item['link_path']) || !menu_valid_path($item)) {
    form_set_error('link_path', t("The path '@link_path' is either invalid or you do not have access to it.", array('@link_path' => $item['link_path'])));
  }
}

Using Drupal 6.20 and CiviCRM 3.3.2

Hi ken,

Does your solution work on versions older than CiviCRM 3.4.x and Drupal 6.22?

I have a project which needs this feature but they are currently on
CiviCRM 3.1
Drupal 6.16
.

Maybe we should upgrade the Drupal and CiviCRM to make the clean URL work?

Do you know other Drupal modules might be incompatible with the URL_Alter ad CiviCRM?

Currently we have Twitter, Views, and Webform modules at the Drupal site.

Thanks!