[imp] Dimp: purged messages

[Michael M Slusarz]

[Please keep discussions on the list]

Quoting Deja3 <deja3-user at bitrealm.com>:

> Quoting Michael M Slusarz <slusarz at horde.org>:
>
>> Quoting Deja3 <deja3-user at bitrealm.com>:
>>
>>> Quoting John Capo <jc at irbs.com>:
>>>
>>>> Quoting Deja3 (deja3-user at bitrealm.com):
>>>>> Same problem for me.  Turning on javascript debugging, I get the
>>>>> following error about 5 seconds after I delete the message:
>>>>>
>>>>> doActionComplete callback: InternalError: too much recursion
>>>>
>>>> Ditto with RC4 and 1.0 release.  Dimp is pretty much dead at that
>>>> point for me.  Deleting works fine with RC3.
>>>
>>> True enough.  I just loaded dimp RC3 and deleting seems to be working
>>> just fine.   RC4 doesn't work for me either (nor the 1.0 release) so
>>> my experience is the same as yours.
>>
>> You may want to look at this commit message:
>> http://lists.horde.org/archives/cvs/Week-of-Mon-20080526/079072.html
>>
>> particularly this patch:
>> http://cvs.horde.org/diff.php/dimp/js/src/DimpBase.js?r1=1.1.2.47&r2=1.1.2.48&ty=u
>>
>> and see if that fixes things for you.  I can't reproduce on HEAD btw.
>
> This particular patch for dimp/js/src/DimpBase.js didn't fix the  
> issue for me; however, I downloaded the latest CVS version of DIMP  
> and it did fix the delete issue.  So, I'm now running Horde 3.2 with  
> Dimp 2.0-cvs and have delete working.
>
> So it does appear that HEAD is working and dimp H3 1.0 is broken.

Did you update the compressed version in dimp/js?  By default, dimp  
doesn't serve the version out of js/src - I just provided that link so  
people could see the changes (providing a URL to a diff for the  
compressed version is pretty much impossible to read).

If not compressing yourself, you need to patch the js/DimpBase.js  
version also and ensure your browser is not serving a cached version.   
As of right now, there are no code changes between the 2 branches so  
if DIMP HEAD works and DIMP FW_3 (to be 1.0.1) doesn't there are  
issues on your installation.

michael

-- 
___________________________________
Michael Slusarz [slusarz at horde.org]