Skip to content

Changed layout of accordions.#2

Merged
beat merged 1 commit intomysqlprofilingfrom
unknown repository
Jun 15, 2013
Merged

Changed layout of accordions.#2
beat merged 1 commit intomysqlprofilingfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jun 15, 2013

Fixed some css issues.
Changed classname syntax.
Added language strings.
Improved method of determining what queries should be marked green/red/yellow.
Added back the 0.3% min width to bar parts.
And more...

Fixed some css issues.
Changed classname syntax.
Added language strings.
Improved method of determining what queries should be marked green/red/yellow.
And more...
@ghost
Copy link
Copy Markdown
Author

ghost commented Jun 15, 2013

You might want to play around with the $timeScore if structure, which determine which queries are marked green/yellow/red.
If is no based on $timeScore:
5+ (red)
2.5 - 5 (yellow)
2.5- (green)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like that there is one space instead of two between the param type and its name.
Could it be that you're using PhpStorm ? This is oh so annoying....

@beat
Copy link
Copy Markdown
Owner

beat commented Jun 15, 2013

@nonumber : Great stuff! Well done, and many thanks of doing all the cleanups.
Merging it as is, as already much better than main branch mysqlprofiling

beat added a commit that referenced this pull request Jun 15, 2013
Merging Changed layout of accordions from @nonumber with many thanks ! Well done!

Actually if you want, you could apply the same kind of bargraphs to the PHP+Memory+Total SQL queries time total in there in the profiling section. That way it would be immediately visible if a badly written small module kills a site's performance. Even more, with correlating the PHP profiling utime-timestamps with the SQL queries ones, the PHP section could be spitting out really useful information. ;)
@beat beat merged commit 342a01c into beat:mysqlprofiling Jun 15, 2013
beat pushed a commit that referenced this pull request Jul 18, 2013
beat pushed a commit that referenced this pull request Feb 17, 2016
Allow users to accept & configure stats sending
beat pushed a commit that referenced this pull request Feb 17, 2016
Add an implementation of JCryptCipher using the Crypto class
beat pushed a commit that referenced this pull request Feb 17, 2016
beat added a commit that referenced this pull request Jan 22, 2022
Fixes `Deprecated: mysqli_real_escape_string(): Passing null to parameter #2 ($string) of type string is deprecated in libraries/joomla/database/driver/mysqli.php on line 254`
beat added a commit that referenced this pull request Jan 23, 2022
…fixes Deprecated tinymce.php on line 603)

Fixes `Deprecated: htmlentities(): Passing null to parameter #2 ($flags) of type int is deprecated in plugins/editors/tinymce/tinymce.php on line 603`
Joomla 3.10 version of Joomla 4.0 joomla#36761
beat added a commit that referenced this pull request Jan 23, 2022
…6787)

* [3.10] [PHP 8.1] Mysqli database driver escapt function fix

Fixes `Deprecated: mysqli_real_escape_string(): Passing null to parameter #2 ($string) of type string is deprecated in libraries/joomla/database/driver/mysqli.php on line 254`

* Fix also the other db drivers

Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
HLeithner pushed a commit that referenced this pull request Jun 27, 2022
…ler-sql-improvements-mod-1

[CMS PR 36506] Branch update and merge conflicts resolution
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants