Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Event export with ZM_USE_DEEP_STORAGE option set #506

Closed
DarthBrader opened this issue Aug 19, 2014 · 2 comments
Closed

Event export with ZM_USE_DEEP_STORAGE option set #506

DarthBrader opened this issue Aug 19, 2014 · 2 comments

Comments

@DarthBrader
Copy link

When the ZM_USE_DEEP_STORAGE option is set, the event export function generates faulty javascript in the zmEventImagesMaster HTML file. It does not use the deep path format, so the links to the actual event data are broken.

The issue appears to be with the file .../skins/classic/includes/export_functions.php. Someone offered a solution on the ZM forums, but it effectively hard-codes deep paths. See http://www.zoneminder.com/forums/viewtopic.php?f=29&t=18573.

This issue has also been reported on Ubuntu on Launchpad. https://bugs.launchpad.net/ubuntu/+source/zoneminder/+bug/1176128.

I am running Ubuntu 14.04.1 LTS and ZoneMinder 1.27, which I compiled from source.
@onlyjob
Copy link
Contributor

onlyjob commented Apr 4, 2015

This issue was reported in Debian as well: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=723706

I confirm it is still exist in v1.28.1.

Apparently this is because export is not using mygetEventPath...

onlyjob added a commit to onlyjob/ZoneMinder that referenced this issue Apr 4, 2015
@onlyjob
skins/classic: fix HTML export with USE_DEEP_STORAGE (Closes: ZoneMin…
d1f00f0 
…der#506).

 Use mygetEventPath() to get valid path.

Signed-off-by: Dmitry Smirnov <onlyjob@member.fsf.org>
@knight-of-ni
Copy link
Member

When this was reported initially, I thought this described an issue with the location on the filesystem where the export data was written. Rather, the issue is with one of the generated html files inside the exported archive file. And it is just zmEventImagesMaster. The other files in the exported archives all point to the correct location. I see now. Merging

knight-of-ni pushed a commit that referenced this issue Apr 4, 2015
Merge pull request #782 from onlyjob/export
d25f7ed 
skins/classic: fix HTML export with USE_DEEP_STORAGE (Closes: #506).
@knight-of-ni knight-of-ni mentioned this issue Nov 8, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@onlyjob @knight-of-ni @DarthBrader