Achievo/ATK - Bug 48 – Hour locking feature fails on Win32 systems

Bug 48 - Hour locking feature fails on Win32 systems

Summary:

Hour locking feature fails on Win32 systems

Status:	RESOLVED FIXED

Product:	Achievo
Component:	Time Registration
Version:	0.8.*
Platform:	All Linux

Importance:	P3 normal
Target Milestone:	---
Assigned To:	Ivo Jansch

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2002-06-27 16:53 by Ivo Jansch
Modified:	2004-11-13 12:43 (History)

Attachments
Proposed new version of weekstamp() function (by Joachim) (1.34 KB, text/plain) 2002-06-27 16:57, Ivo Jansch	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Ivo Jansch 2002-06-27 16:53:27

Symptom, as described by Joachim Nubold:

"After weeks have been locked (for all), people still can book
additional hours or change their existing bookings"

------- Comment #1 From Ivo Jansch 2002-06-27 16:54:09 -------

Additional comment plus possible patch from Joachim:

"think I got it. On win32 systems, at least with some PHP versions, not
all strftime functions are working. This results in week numbers not
being recognized in Achievo. 
For anyone having the same problem, I have attached the fix that works
for me. Anyone intending to use, check carefully, since I'm not that
used to PHP."

------- Comment #2 From Ivo Jansch 2002-06-27 16:57:55 -------

Created an attachment (id=6) [details]
Proposed new version of weekstamp() function (by Joachim)

------- Comment #3 From Ivo Jansch 2003-11-10 22:46:23 -------

I'm hesitant to put this into 1.0, since it might break stuff for other users.
I'll leave it like it currently is for now, and point any user that experiences
this problem to this bug, to test the patch.

------- Comment #4 From Ivo Jansch 2004-01-13 08:37:09 -------

Version cleanup.

------- Comment #5 From Ivo Jansch 2004-01-15 13:32:30 -------

Christophe Cornil mentioned on the mailinglist that the patch worked for him
too. 

I'll add the patch to Achievo 1.0.3.

------- Comment #6 From Ivo Jansch 2004-11-13 12:43:09 -------

The fix is now committed to CVS, so as of 1.1.RC3, the weekstamp method should
work on both windows and unix servers.