[Phpmyadmin-devel] Github Fork+Clone = PHP Error. Also Status Monitor.
rouslan at placella.com
Tue Dec 4 14:25:19 CET 2012
On 04/12/12 08:53, Tyron Madlener wrote:
> > There are several other things that aren't right with the server status
> > pages. In general I find that there are many problems related to the
> > tabbed navigation, so I'm splitting the different server status features
> > into separate pages. This should also improve further maintainability of
> > the system.
> Oh yes. That's probably a good idea.
> > By the way, will you be hacking the server status pages any time soon?
> > We might want to try and avoid any merge conflicts, if you do...
> The only thing I might do is fix some of the bugs and glitches that I
> notice, which however should be only very local changes. If I do, I'll
> make it as a pull request on github. I'll leave a note here if I plan to
> make any bigger changes :-)
Feel free to directly assign any pull requests relative to the server
status pages to me. I'll look into integrating those with my changes.
> On Mon, Dec 3, 2012 at 11:45 PM, Rouslan Placella <rouslan at placella.com
> <mailto:rouslan at placella.com>> wrote:
> On 03/12/12 18:39, Tyron Madlener wrote:
> > Sadly there also seems to be a big memory leak with jqplot.
> Letting the
> > monitor run for a longer time causes my Chrome consume 1-2 GB of
> > Maybe old data values are not being deleted?
> > On Mon, Dec 3, 2012 at 6:29 PM, Tyron Madlener <tyronx at gmail.com
> <mailto:tyronx at gmail.com>
> > <mailto:tyronx at gmail.com <mailto:tyronx at gmail.com>>> wrote:
> > Heya. Haven't written in a while. This appears when I browse
> a table
> > from my forked github pma of today:
> > *Parse error*: syntax error, unexpected T_PAAMAYIM_NEKUDOTAYIM in
> > line *3879*
> > The seems to be a lot of new bugs + glitchy behavior in the
> > monitor, since the transition to jqplot. Here are a few i
> could pin
> > down:*
> > - roccivic for some reason deletes the local storage when the
> > unloads, though it's intention was for storing settings:
> Yep, my bad. I'll put that back some time soon.
> > - Reloading the page while in the monitor will result in a js
> > in jqpload and not load the charts
> > - The query analyzer dialog doesn't show the profiling chart
> > anymore. For this however I took the freedom to fix this +
> > glitches myself and send a pull request on github:
> > https://github.com/phpmyadmin/phpmyadmin/pull/113
> There are several other things that aren't right with the server status
> pages. In general I find that there are many problems related to the
> tabbed navigation, so I'm splitting the different server status features
> into separate pages. This should also improve further maintainability of
> the system.
> By the way, will you be hacking the server status pages any time soon?
> We might want to try and avoid any merge conflicts, if you do...
> > On another note:
> > - Mousewheel scrolling doesn't work in the left pane when I
> > 1. Expand a DB with many tables
> > 2. Expand the tables
> > 3. Switch to page 2
> > - Also the left pane scroll bar is so massively fat, is that
> Yes, that's how I intended it to be. JM, had provided a thinner
> replacement version, but that also came with a list of bugs that IMO
> didn't justify the benefits.
> > Sorry for throwing these bug reports out so chaotically.
> Thanks for
> > reading anyway!
More information about the Developers