Closed (fixed)
Project:
Views 404
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
19 Apr 2011 at 08:55 UTC
Updated:
20 May 2013 at 22:20 UTC
Jump to comment: Most recent file
Hello,
I have just added a quick fix which seems to be working on Drupal 7.
I have committed it to the git repository.
Basically, it is only modifying core=6.x into core=7.x.
If I find any problems, I'll trying posting a patch.
Bye.
| Comment | File | Size | Author |
|---|---|---|---|
| #16 | Issue-1131834-by-makangus-hass-D7-Port.patch | 2.03 KB | hass |
| #15 | Issue-1131834-by-makangus-hass-D7-Port.patch | 2.02 KB | hass |
| #8 | views404-d7-port-1131834-8.patch | 2.19 KB | makangus |
| #1 | 1131834-drupal7.x-1.patch | 262 bytes | fe_lix_ |
Comments
Comment #1
fe_lix_ commentedHere is the patch file.
Comment #2
basicmagic.net commentedsubscribe
Comment #3
chalk commented@fe_lix_ Your patch work, but after installing I have messages "Notice: Undefined index". To fix them find in views404.module 13 line and replace:
Comment #4
gpmaker commentedThank you @Chalk, it work like a charm =)
Comment #5
gpmaker commentedGuys let's post the version for 7.x
Comment #6
kalis1Marked #908922: drupal 7 port or Views 6.x/7.x as duplicate of this issue.
Comment #7
bago commentedI'm using this (#1 + #3) and I'm very happy.
I found that ajax preview in views 3.4/3.5 is broken by this module so I added a specific check in the pre_view hook (lines starting with +):
We should really make this code into git and have a proper 7.x release of this useful module!
Comment #8
makangus commentedRolled a patch with @fe_lix_, @Chalk and @bago's input with a few things to fix some PHP notices.
Comment #9
plazik commentedThanks makangus.
It works for me (patch for dev version).
It's time for 7.x-dev version.
Comment #10
makangus commentedThe patch #8 is against dev.
Comment #11
eule commentedwork this with views 3 ??
Comment #12
plazik commentedYes, it works.
Comment #13
eule commentedhi plazik
which patch you applied?
Comment #14
plazik commentedPatch at #8 http://drupal.org/node/1131834#comment-6461198 to dev version.
Comment #15
hass commentedPatch in #8 does not apply here against latest DEV. Reroled.
Comment #16
hass commentedAdded package Views.
Comment #17
hass commentedComment #18
eule commentedthx hass, plazik
switching to new theme and have a lot to reorganize...test it in a couple of days
Comment #19
hass commented