Comments (7)
How are you calling Krumo that you're getting this output?
from krumo.
I tried both the global function and the object based method: I only receive the error when using the global function.
from krumo.
What were you trying to dump? Line 532 is a pretty weird line to die on. Line 532 is checking if you've disabled Krumo globally.
What version of PHP are you running?
from krumo.
Hey sorry for my late reply (had some time off).
We're running php 5.6.
I'm dumping get_defined_vars()
not sure if that might be a problem?
Krumo doesn't die either, if we weren't running raygun I would have never noticed anything.
I just wanted to let you know that raygun keeps throwing these errors at me.
from krumo.
I'm guessing raygun ignores the @ error operator (didn't just found out about these).
I'll try with raygun, they should ignore these errors.
from krumo.
from krumo.
@scottchiefbaker + @mmucklo Perhaps the code can be refactored not to have the silence operator at all? Right now it is
if (in_array($function, array("krumo","k","kd")) || (strToLower(@$d['class']) == 'krumo') || (is_callable($callback) && call_user_func($callback, $d))) {
break;
}
Let's break it up into 3 separate statements with an extra check if the "class" element is not empty, like this:
if ( in_array($function, array("krumo", "k", "kd")) ) {
break; // krumo alias function
}
if (!empty($d['class']) && ('krumo' == strToLower($d['class'])) ) {
break; // krumo class method
}
if ( is_callable($callback) && call_user_func($callback, $d) ) {
break; // callback
}
It's more readable this way with each case for exiting the loop clearly defined, AND with the extra check that $d['class']
is not empty.
from krumo.
Related Issues (20)
- Escape <!-- and I suppose --> HOT 6
- krumo provide more info on a array-casted object than on the object itself HOT 4
- Feature request (and patch) : shift+click to expand all HOT 8
- Javascript bug in version 0.5.6 HOT 4
- Non-static method called statically HOT 14
- Typo in README.md "Usage" section example HOT 1
- Don't truncate after 50 characters HOT 5
- Update Packagist? HOT 3
- How to change skin? HOT 17
- krumo not showing private and protected properties HOT 2
- Is it possible to 'queue' so they can be displayed in bulk at once HOT 2
- Dropping VERSION file in favor of a class constant HOT 1
- Using a separate method for returning Krumo dumps instead using KRUMO_RETURN as argument HOT 1
- Drop the INSTALL file HOT 1
- Put composer.lock in .gitignore and drop it HOT 3
- Drop "@static" and "@access public" HOT 2
- Drop KRUMO_DIR, replace with __DIR__ HOT 1
- Change how configuration is operated HOT 2
- Krumo doesn't cleanly support PHP 8.1 HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from krumo.