Tried to input an internal path with a query attached, it did not attach the trailing query.

CommentFileSizeAuthor
#2 linkit-input-url-parsing-2552773-2.patch529 bytesashalan

Comments

ashalan created an issue. See original summary.

ashalan’s picture

StatusFileSize
new529 bytes

The problem is that this input method's paths are not run through the url method in

function linkit_get_insert_plugin_processed_path(LinkitProfile $profile, $uri, $options = array()) {
    switch ($profile->data['insert_plugin']['url_method']) {
      case LINKIT_URL_METHOD_RAW:
        $path = $uri;
        break;
      case LINKIT_URL_METHOD_RAW_SLASH:
        $options['alias'] = TRUE;
        $path = url($uri, $options);
        break;
      case LINKIT_URL_METHOD_ALIAS:
        $path = url($uri,  $options);
        break;
    }

    return $path;
}

.

Attached patch should fix this issue. I am unsure as to why this was the case for this input method.

ashalan’s picture

Status: Active » Needs review
ashalan’s picture

Assigned: ashalan » Unassigned
henrijs.seso’s picture

Status: Needs review » Needs work

Works. Needs re-roll against dev version.

henrijs.seso’s picture

Version: 7.x-3.4 » 7.x-3.x-dev
anon’s picture

Status: Needs work » Closed (cannot reproduce)

Raw paths it not run through the url method because it will create an alias. Raw paths is just raw i.e. not processed at all.

And I can't reproduce this.

I added a link node/1?test=1 and that works.