Closed (fixed)
Project:
Diff
Version:
7.x-3.0-alpha1
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
27 Feb 2012 at 15:48 UTC
Updated:
22 May 2013 at 11:57 UTC
Jump to comment: Most recent file
Comments
Comment #1
duaelfrThis patch applies on the current 2.x-dev branch
It is very close to #248778-14: Taxonomy diff
Comment #2
duaelfrComment #3
duaelfrDO NOT USE THESE FILES !
Comment #4
duaelfrDO NOT USE THESE FILES !
(I am really tired sorry)
Comment #5
duaelfrFinally !
Two patch files fixing a little bug occuring when a field have a cardinality of 1 and is empty.
Comment #6
alan d. commentedMassive patch over in #1365750: Generalize API and Integrate with core field types needs some testers. If accepted, this will make this issue a duplicate.
Comment #7
alan d. commentedIn another task, I'm refactoring the field comparison.
These are two raw outputs based on the minimal comparison and the full comparison for images, files and lists (will cross post this to #1550698: Diff of "select from list" fields shows change in key, not change in value if you want to comment on the list format). Currently the output is controlled by a system variable that defines the defaults.
Selectable components are separated with a semi-colon, but the file name is not configurable.
Comment #8
alan d. commentedThis should be resolved with the new branch, 7.x-3.0-alpha1 or 7.x-3.0-dev.
This is still alpha, so test carefully before use on a production site and report any issues back to the queues.
Comment #10
ts145nera commentedHi,
thank you your works.
I've a question: old files are accessible also if revision isn't public.
There's a way to deny access to the old files?
Comment #11
alan d. commented@ts145nera
Can you confirm that these are not assessable without diff? The module does run the fields though an access check.
Note that this check was not present in early 3.x releases, and completely absent from 2.x branch.
Comment #12
ts145nera commented@Alan D.
Thank you for your answer.
If I try without Diff, old files are deleted.
Only with Diff old files are stored.
When I access to the node, all work fine and I can see only last file (public files)
But If I try to access to the old file by the Url, the access is possible.
I set 700 permissions to the directory, but it doesn't work. Some search engine link in their cache system, so it can remember link to the old file (es. Google cache)
There's a way to prevent this behavior?
Comment #13
alan d. commentedWith Diff or in general when you save a revision? I'm baffled to understand how Diff would interact with the revision system like this, although you need to save revisions (Drupal core functionality) to get Diff to work comparing revisions.
Are you running private or public file system? Core should block access to revision-only files by default if the user does not have access to the revision itself. If public, then no other solutions don't come to mind at the moment sorry, bar turning off revisioning and thus effectively disabling Diff itself on this content type. :(
Comment #14
ts145nera commented@ Alan D.
I'm really sorry Alan, yesterday I forgot to check "create new revision" when I change attached files.
Also without Diff, if I try to create new revision the problem remain.
I'm running public file system, but core doesn't block file access.
You can reproduce problem following these steps:
1) create content type with file field
2) add node and upload one file (copy link to the file http://...........txt)
3) change node deleting file and add another one checking "create new revision"
4) now you can access to the first uploaded file by the copied link.
Comment #15
alan d. commentedIf not too late, change to private file system and use "View content revisions" permission to restrict. Good luck!
Comment #16
ts145nera commentedThank you Alan D.
You're right, private file system is the only way to restrict access.
I'll try to change public to private file system.