I love this module, and would love it even more, if I could get it to work with contextual views.

Im trying to 'take over' this page-view: /question/tagged/% , where '%' is the tax-term in context. I think the module gets confused by the '%'-wildcard!?

When I flush the cache I get following notice: 'questions/tagged/% has not been overridden because the menu item does not exist.' , which pretty much confirm, that its not possible to override wildcard-paths.

Can someone confirm this? - Thanks.

CommentFileSizeAuthor
#7 1574574-7.patch783 bytesswentel

Comments

swentel’s picture

swentel commented

that's probably not possible, but it's easier to use the 'Views content' module for it I think, it comes with more configuration options once you add a pane.

sbilde’s picture

sbilde commented

Im not familiar with that module called Views Content, and can't seem to locate here on drupal.org?

swentel’s picture

swentel commented

That comes with ctools itself, look on the modules page in the fieldset of Ctools.

sbilde’s picture

sbilde commented

Okay. Actually, the additional content that I wanted to show in the sidebars of the page, was in fact some 'views content panes' with various content along with the 'existing node', which originally belonged to the menu, that was 'taken over'.

Maybe I'ts easier just to create a panel-page along with a taxonomy-variant, and set up some contextual selectors!?

It would be really awesome, if this module could handle 'wildcard'-paths in the future..- PM EXISITNG PAGES, FTW :). - Once again thank you, for your great contributions Swentel.

andypost’s picture

andypost
he/him
Primary language Russian
commented

handle 'wildcard'-paths in the future

it does for now but there's some troubles with #1440068: Wrong usage of arguments

swentel’s picture

swentel commented

The patch here http://drupal.org/node/1440068#comment-5998798 will probably make this work. Try overriding the /question/tagged path - so without the %. If that works, I'll update the documentation for this on the project page and on the configuration screen.

swentel’s picture

swentel commented
StatusFileSize
new 1574574-7.patch783 bytes

Ok, the problem is the wildcard. For views, this should be %views_arg instead of %. Patch attached, tested and committed.

swentel’s picture

swentel commented
Status: Active » Fixed
sbilde’s picture

sbilde commented

Beautiful! The little snippet from #7, along with the %views_arg -element did the trick. - This module has just upgraded to be a permanent part of my tool belt (make-file)...- Really powerful!

- You rock Swentel ;)

andypost’s picture

andypost
he/him
Primary language Russian
commented

Suppose this needs some clean-up

+++ b/plugins/export_ui/pm_existing_pages.inc
@@ -162,6 +162,9 @@ function pm_existing_pages_determine_path($lf, $path) {
+        $parts[$position] = $parts[$position] = '%views_arg';

What this code means? Looks like you assign the value twice.

swentel’s picture

swentel commented

You're right, never saw those double declarations, removed them in http://drupalcode.org/project/pm_existing_pages.git/commit/e8a7ce1

Status: Fixed » Closed (fixed)

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