[Phpmyadmin-devel] GSoC - Removal of frames

Dieter Adriaenssens dieter.adriaenssens at gmail.com
Thu Jun 7 21:18:30 CEST 2012


2012/6/4 Dieter Adriaenssens <dieter.adriaenssens at gmail.com>:
> 2012/6/4 Rouslan Placella <rouslan at placella.com>:
>> On 03/06/12 21:39, Dieter Adriaenssens wrote:
>>> Hi Rouslan,
>>>
>>> I went through your latest commits in your GSoC branch, see github for
>>> the small remarks.
>>>
>>> You did a great job on creating the different classes (PMA_Header,
>>> PMA_Footer, PMA_Response, ...) moving functionality from other library
>>> files into it, and integrating these new classes in the current code
>>> base.
>>>
>>> I think you've come to a point where these changes are ready to be
>>> merged with master. What do you think?
>>>
>>
>> Hi Dieter,
>>
>> I'd love to merge, but I feel that my branch isn't quite ready. I have
>> an issue with the error_handler class where errors are generated while
>> displaying previous errors. The number of errors increases exponentially
>> at every page load until eventually the session explodes. So, yeah, I
>> wouldn’t dare merging something as bad as this into master.
>
> Hi Rouslan,
>
> Good thinking! :p
>
> Let me know when you found and fixed that bug.

Hi Rouslan,

I read in the weekly report of Alex Marin (another GSoC student) that
he will start working on the authentication files soon. I noticed you
made some changes to these files as well, so it's a good idea to
discuss this before the start of the merge.

@Alex Marin: have you started working on these files? When do you plan
to start? It seems like a good idea that you start working on the
version with the commits of Rouslan, to avoid merge problems later on.

Kind regards,

Dieter Adriaenssens




More information about the Developers mailing list