Forum OpenACS Development: Re: Notifications of File-storage

12: Re: Notifications of File-storage (response to 1)

Posted by Nima Mazloumi on 06/18/04 08:28 PM

Tilmann...if I understand you correctly we already have notification for 5.1 or only a patch that needs to make its way to the source?

13: Re: Notifications of File-storage (response to 12)

Posted by Tilmann Singer on 06/18/04 08:52 PM

Nima - only a patch that didn't make its way to the source in time apparently (here https://openacs.org/bugtracker/openacs/com/file-storage/patch?patch_number=146 - linked from the bug posted above).

14: Re: Notifications of File-storage (response to 13)

Posted by Orzenil Silva Junior on 06/18/04 10:18 PM

Hi
This patch didn't aplly cleanly oacs-5-1. I tried to do it in a fresh install and didn't work fine.
I change manually some failed hunks and I think this patch works ok now. I tried it in
http://oacs-cf.intertexto.info:8002/noticias-arquivos/
You can see Request notification link and it works - I received an email by notification service :)

Notification for: File-Storage: File deleted Name of the File deleted: imagem1 View folder contents: http://oacs-cf.intertexto.info:8002/noticias-arquivos/?folder_id=1233

I'm trying make a new patch file and upload it to bug tracker.
/Orzenil

15: Re: Notifications of File-storage (response to 13)

Posted by Orzenil Silva Junior on 06/19/04 12:39 AM

I submitted a slight modified patch file to add notifications to oacs-5-1-0 file-storage package.
There are no great changes from patch #146 submitted by Abhina Peddada.

You see it at https://openacs.org/bugtracker/openacs/patch?patch_number=534

16: Re: Notifications of File-storage (response to 15)

Posted by Tilmann Singer on 06/19/04 04:52 PM

Patch applied to HEAD, thanks. If anyone has time please test on oracle. And if anyone has even more time, i18n-ize it ...