<html>
<head>
</head>
<body>
<br>
<br>
Marc Delisle wrote:<br>
<blockquote type="cite" cite="mid:3D4BF796.8020801@CollegeSherbrooke.qc.ca"><br>
<br>
Robin Johnson wrote:<br>
<blockquote type="cite" cite="mid:Pine.LNX.4.43.0208030122310.23247-100000@fermi.orbis-terrarum.net">
<pre wrap="">On Fri, 2 Aug 2002, Robin Johnson wrote:<br></pre>
<blockquote type="cite">
<pre wrap="">If you have data in a table, and hit browse, the SQL query box has 3<br>links. "[Edit] [Explain SQL Code] [Create PHP Code]" however, if I paste a<br>query into the SQL box, such as "Update PMA_relation SET<br>master_db='foobar'", the result box from that does not have the same set<br>of links. Could somebody help me figure out why ASAP? This is important<br>for 2.3.0, as well as my surprise feature (see below).<br></pre>
</blockquote>
<pre wrap=""><!---->I found out some of this, and cleaned up a lot of the code in the area,<br>plus made it more customizable via $cfg, and fixed a bug with the EXPLAIN<br>I noticed (You could hit EXPLAIN, but not go back from there).<br><br>I'm closer to the bug, but still working on it.</pre>
</blockquote>
To see the links, uncheck "Show this query here again". The logic in common.lib
line 1136<br>
does not display the links. I wonder why.</blockquote>
I think it was to avoid displaying the "SQL-query: Edit" link when the query
is already displayed.<br>
But the IF line 1136 should only control the generation of $edit_link and
$edit_target, and there <br>
should be 4 IFs at line 1232 to display each link.<br>
<blockquote type="cite" cite="mid:3D4BF796.8020801@CollegeSherbrooke.qc.ca"><br>
<blockquote type="cite" cite="mid:Pine.LNX.4.43.0208030122310.23247-100000@fermi.orbis-terrarum.net">
<pre wrap=""><br><br></pre>
<blockquote type="cite">
<pre wrap="">On a last note, I'm just about ready to merge my surprise feature, just<br>tidying it up for PHP clean-ness myself now, but I can see quite a few<br>people liking it, and it possibly gaining us a lot of respect in the<br>actual SQL development fields. The only catch is that it has a fairly high<br>requirements list (PHP XML extension + a few PEAR modules), so it is<br>disabled by default in the configuration file, with notes that it needs a<br>lot of things to work.<br></pre>
</blockquote>
<pre wrap=""><!---->Partial merge is starting, completely non-intrusive.<br><br></pre>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>