Skip to forum content

LuxCal Web Calendar

The place for news, questions and suggestions

You are not logged in. Please login or register.

Post new reply

Post new reply

Compose and post your new reply

You may use: BBCode Images Smilies

All fields with bold label must be completed before the form is submitted.

Required information for guests


Required information

Topic review (newest first)

5 Post by remat457

Ohh, toolbox.php smile

Fix works great btw, thanks!

4 Post by Roel

Good guess, but not quite right wink
Thanks for reporting this problem.

I've posted a fix in this forum under Known Issues and Fixes.

Roel

3 Post by remat457

Awesome, thanks!
I think I saw that variable in messaging.php and was hoping it was global. I was php-guessing lol

2 Post by Roel

Good point.

And since you've already given me the file and line number, it should be a piece of cake wink However, to make the attachment accessible from an email reminder, the change should be made somewhere else. I will see what should be done.
Where did you get this $calUrl? This is not an existing global variable, it's only used locally in the file messaging.php. We should use $set['calendarUrl'] and strip off the possible suffix "?cal=<calID>".
I will add this change to the next calendar version 4.7.1 and tomorrow, Belgium time, I will post a fix, for those who can't wait smile

Roel

1 Post by remat457

I noticed that the email messages with attachments that the attachment link points to relative path: ./attachments

so they cannot be clicked and opened (from the email).

What do you think about changing eventform1.php (line 106) from ./attachments to:
{$calUrl}/attachments ?

I do suspect there would be a performance hit clicking on the attachment from the actual calendar.
I am not actually PHP-fluent, so I thought I would ask before trying it first smile

Thanks