Are there allready plans for a D7 port?
Did anybody allready started working on this?

CommentFileSizeAuthor
#13 view_url_fix-939002-13.patch948 bytesrobeano
#3 views_savedsearches_d7.patch29.9 KBattiks
#2 939002_views_savedsearches_d7_3.patch26.91 KBattiks
#1 views_savedsearches_d7.patch26.6 KBattiks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

attiks’s picture

attiks CreditAttribution: attiks commented
Status: Active » Needs work
FileSize
views_savedsearches_d7.patch26.6 KB

Included a patch to convert the module to D7, notifications module is not ported!

attiks’s picture

attiks CreditAttribution: attiks commented
Status: Needs work » Needs review
FileSize
939002_views_savedsearches_d7_3.patch26.91 KB

New patch, removed dpm calls and updated schema

attiks’s picture

attiks CreditAttribution: attiks commented
FileSize
views_savedsearches_d7.patch29.9 KB

New version attached:
- includes fix for #973178: Support views in code
- has a new permission to view all saved searches
- database schema is altered, added new column `view_name` VARCHAR(32)

skeeve’s picture

skeeve CreditAttribution: skeeve commented

does anybody have working patched d7 version? give me pls, can`t apply patches (

Jelle_S’s picture

Jelle_S
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: Jelle_S commented

Sandbox project containing the drupal 7 code for Views Saved Searches was created here.

skeeve’s picture

skeeve CreditAttribution: skeeve commented

thx!

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented

cool!

dgastudio’s picture

dgastudio CreditAttribution: dgastudio commented

subscribe

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Threre's one thing that's missing in this patch: you need to remove the 'module' and 'theme' arguments from drupal_add_js in lines 667 and 668 or else these won't be included in admin interfaces, breaking collapsed fieldsets.

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
Title: D7 port » D7 port of Views Saved Searches

;-)

Jelle_S’s picture

Jelle_S
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: Jelle_S commented

@kevee: Thanks! updated in the sandbox ;-)

PedroMiguel’s picture

PedroMiguel CreditAttribution: PedroMiguel commented

will notifications_views_savedsearches also be ported?

robeano’s picture

robeano CreditAttribution: robeano commented
FileSize
view_url_fix-939002-13.patch948 bytes

I was getting a Fatal error (about using $view->get_path()) when I tried to save the view. The view was saved, but the URLs for each saved search were invalid. The savedsearches urls were missing the view path but did have the correct query string.

The attached patch fixes these errors and should apply cleanly to the sandbox project (which is what I've been working with: http://drupal.org/sandbox/jelles/1079940

Jelle_S’s picture

Jelle_S
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: Jelle_S commented

Patch applied to the sandbox project

designcontext’s picture

designcontext CreditAttribution: designcontext commented

This module seems to be exactly the one I'm looking for. Are there any plans for a port to d7, even a dev-version?
Thanks!

attiks’s picture

attiks CreditAttribution: attiks commented

For the moment use the sandbox at http://drupal.org/sandbox/jelles/1079940

designcontext’s picture

designcontext CreditAttribution: designcontext commented

Hm, never worked with git and co. So I must wait... thanks for your quick reply!

attiks’s picture

attiks CreditAttribution: attiks commented

Keep an eye on #1677172: Offering to co-maintain Views Saved Searches, will be fixed one of these days.

designcontext’s picture

designcontext CreditAttribution: designcontext commented

Thanks for making this an official dev-version!
Unfortunately the D7-version does not work for me. I can save the search. The search is listed in a fieldset. But: The link of the saved search doesn't work, the original view is linked without any queries. And deleting the search doesn't work.

attiks’s picture

attiks CreditAttribution: attiks commented
Version: 6.x-1.x-dev » 7.x-1.0-beta1
Assigned: Unassigned » Jelle_S
Status: Needs review » Active
Jelle_S’s picture

Jelle_S
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: Jelle_S commented

The delete functionality was broken indeed, but all the rest seems to work fine.
Delete fixed in latest dev.

Could you provide me steps to reproduce?

designcontext’s picture

designcontext CreditAttribution: designcontext commented

I tested the new dev-version, but deleting stil doesn't work for me.
After saving the search I have this link:

<label class="option" for="edit-saved-searches-2">
<a href="/musiker" class=" active">fortgeschritten</a>
</label>

The path "musiker" is the path of the view.
What infos do you need?

Sam152’s picture

Sam152 CreditAttribution: Sam152 commented

It looks like there is a D7 version as an official release on this module. I know for a fact the notifications integration has not been ported yet, but how does the status of this version compare to the sandbox module?