[Bitpim-devel] PM-8200 bugs (moved from bitpim-user)

[Stephen Wood]

I posted some regarding call alarms that I haven't received replies to. They
* Writing call alarms to the phone works, but subsequent writes add
duplicates to those in the phone rather than replacing the existing ones.
* I don't remember for sure now, but I think writing a call alarm to the
phone also wrote a matching regular event.

Jim:

I can't reproduce these on a SCP-5500.

Can you verify that the PM-8200 has 15 call alarm slots. (That is what
the manual says and BitPim assumes.)

Could you describe a sequence of operations that can reproduce this
bug, starting with a phone with no events or call alarms on it.

Thanks, Stephen

[Jim Smith]

* Writing call alarms to the phone works, but subsequent writes add
duplicates to those in the phone rather than replacing the existing ones.
I can't reproduce these on a SCP-5500.
Can you verify that the PM-8200 has 15 call alarm slots. (That is what
the manual says and BitPim assumes.)
Thanks, Stephen

Now I can't reproduce it either with the CVS version. The phone itself says
it has 15 call alarm slots, but when I could reproduce the multiplying call
alarms, BitPim's log showed it writing call alarms into the 20's, even
though the phone said there were 15 slots used and 0 available.

Jim

[Jim Smith]

There's some slight weirdness with the 'a' or 'p' displaying on the BitPim
calendar screen. If I have an existing event and add a new event or call
alarm through BitPim, the one with the latest start time doesn't display 'a'
or 'p' on the screen. The display is correct in the Edit Calendar window.

I also noticed a minor typo in the log. It comes from line 88 in
com_brew.py. It says, "self.log("Reseting phone")" and should say,
"self.log("Resetting phone")". ;^)

Any luck with the deleting call alarm exception? Here's an exception log for
that:

10:10:15.750 Exception: An unexpected exception has occurred.
Please see the help for details on what to do.

Traceback (most recent call last):
File "D:\Projects\BitPim\bpcalendar.py", line 694, in OnDeleteButton
self.cw.DeleteEntry(entry)
File "D:\Projects\BitPim\bpcalendar.py", line 145, in DeleteEntry
del self._data[entry['pos']]
KeyError: 100

Variables by last 8 frames, innermost last

Frame ? in bp.py at line 79
profile = <function profile at 0x007CDA70>
__file__ = 'bp.py'
__name__ = '__main__'
__doc__ = 'Main entry point to Bitpim\n\nIt invokes BitPim in gui
or commandline mode as a

Frame run in D:\Projects\BitPim\gui.py at line 350
args = (['bp.py'],)
m = <gui.MainApp; proxy of C++ wxPyApp instance at
_90cf7400_p_wxPyApp>

Frame MainLoop in D:\Python23\Lib\site-packages\wx\_core.py at line 5262
self = <gui.MainApp; proxy of C++ wxPyApp instance at
_90cf7400_p_wxPyApp>

Frame MainLoop in D:\Python23\Lib\site-packages\wx\_core.py at line 4732
args = (<gui.MainApp; proxy of C++ wxPyApp instance at
_90cf7400_p_wxPyApp>,)
kwargs = {}

Frame OnDeleteButton in D:\Projects\BitPim\bpcalendar.py at line 694
res = 1
newpos = None
enum = 0
entry = {'repeat': None, 'end': [2004, 9, 8, 14, 0],
'description': '2146887088', 'alarm
self = <bpcalendar.DayViewDialog; proxy of C++ wxDialog instance
at _58be1202_p_wxDialo
_ = <wx._core.CommandEvent; proxy of C++ wxCommandEvent
instance at _0cef1200_p_wxCo

Frame DeleteEntry in D:\Projects\BitPim\bpcalendar.py at line 145
entry = {'repeat': None, 'end': [2004, 9, 8, 14, 0],
'description': '2146887088', 'alarm
self = <bpcalendar.Calendar; proxy of C++ wxPanel instance at
_f8951402_p_wxPanel>

[Roger Binns]

Post by Jim Smith
There's some slight weirdness with the 'a' or 'p' displaying on the
BitPim calendar screen. If I have an existing event and add a new
event or call alarm through BitPim, the one with the latest start
time doesn't display 'a' or 'p' on the screen. The display is correct
in the Edit Calendar window.

That is correct by design. The first event in the list that is an
'am' shows the 'a' and the first that is in the 'pm' shows
a 'p'. Otherwise it looks horrible having every event with an
'a' or 'p' after it.

Post by Jim Smith
I also noticed a minor typo in the log. It comes from line 88 in
com_brew.py. It says, "self.log("Reseting phone")" and should say,
"self.log("Resetting phone")". ;^)

Stephen is normally the spelling police :-) I am merely the source
of spelling like that.

Post by Jim Smith
Any luck with the deleting call alarm exception? Here's an exception

You should be glued to the cvs checkins list :-)

Roger

[Stephen Wood]

Post by Jim Smith
Any luck with the deleting call alarm exception? Here's an exception log for

I just committed a fix for that. The call alarms were not being passed
to BitPim correctly for Sanyo phones. You will need to re read the
calendar from your phone.

Stephen

[Jim Smith]

Post by Roger Binns
That is correct by design. The first event in the list that is an
'am' shows the 'a' and the first that is in the 'pm' shows
a 'p'. Otherwise it looks horrible having every event with an
'a' or 'p' after it.

I wouldn't say it "looks horrible", after all, it's only one extra letter.
It looks like you're putting an extra space in there anyway to make things
line up. I vote for showing them all.

Post by Roger Binns

Post by Jim Smith
I also noticed a minor typo in the log. It comes from line 88 in
com_brew.py. It says, "self.log("Reseting phone")" and should say,
"self.log("Resetting phone")". ;^)

Stephen is normally the spelling police :-) I am merely the source
of spelling like that.

Fixed, thanks!

Post by Roger Binns

Post by Jim Smith
Any luck with the deleting call alarm exception? Here's an exception

You should be glued to the cvs checkins list :-)

I am, and this now works too. Thanks to your tips, I was even able to use
the graphical CVS check out this time. :-)

I'll keep watching for the "None" string fix.

[Roger Binns]

Post by Jim Smith
I'll keep watching for the "None" string fix.

I just committed it.

Roger

[Jim Smith]

Post by Stephen Wood

Post by Jim Smith
Any luck with the deleting call alarm exception? Here's an exception log

for

Post by Stephen Wood
I just committed a fix for that. The call alarms were not being passed
to BitPim correctly for Sanyo phones. You will need to re read the
calendar from your phone.
Stephen

I checked out the code and replied to Roger this morning that it does work
fine.

Thanks,
Jim

[Jim Smith]

Post by Roger Binns

Post by Jim Smith
I'll keep watching for the "None" string fix.

I just committed it.
Roger

Does it take a while for committed files to show up, or am I doing something
wrong again? I was able to get updates this morning, but now I get the
following success messages, but no files, when trying to check out and when
trying to update:

When checking out:
~~~~~~~~~~~~~~~~~~
In D:\Projects: C:\Program Files\TortoiseCVS\cvs.exe -q checkout -P bitpim
CVSROOT=:pserver:***@cvs.sourceforge.net:/cvsroot/bitpim

cvs checkout: Empty password used - try 'cvs login' with a real password


Success, CVS operation completed

When updating:
~~~~~~~~~~~~~~
In D:\Projects\BitPim: C:\Program Files\TortoiseCVS\cvs.exe -q update -d -P
CVSROOT=:pserver:***@cvs.sourceforge.net:/cvsroot/bitpim

cvs update: Empty password used - try 'cvs login' with a real password


Success, CVS operation completed

[Roger Binns]

Post by Jim Smith
Does it take a while for committed files to show up, or am I doing
something wrong again?

http://article.gmane.org/gmane.comp.mobile.bitpim.user/1434

Search for "hours". Note who the message was in response to :-)

Roger

[Jim Smith]

Post by Roger Binns
http://article.gmane.org/gmane.comp.mobile.bitpim.user/1434
Search for "hours". Note who the message was in response to :-)
Roger

Dadgumit, you got me again. Sorry 'bout that, I'll check it out in the
morning.