Task #6591

Refactor F3\FLOW3\Log\Backend\FileBackend : getFormattedVarDump

Added by Lukas Lentner over 11 years ago. Updated over 9 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
-
Category:
Log
Target version:
-
Start date:
Due date:
% Done:

50%

Estimated time:
Sprint:
PHP Version:
Has patch:
No
Complexity:

Description

Hi,

I did (my first) refactoring with this function.

Now:
  • Smaller
  • Less control structures,
  • Logic outlines the similarities of an array and an object (to nest) -> I think easier to understand and more consistent

The 'big' change is in fact that I add afterwards the keys of the array/names of the properties. Before we looked after every possibility extra!

A Question:
Why this limitation with the 100 spaces?

Files

6591.diff (2.62 KB) 6591.diff Lukas Lentner, 2010-02-24 18:52
#1

Updated by Lukas Lentner over 11 years ago

Here the diff

#2

Updated by Robert Lemke over 11 years ago

  • % Done changed from 100 to 50
#3

Updated by Robert Lemke about 11 years ago

  • Status changed from New to Needs Feedback
#4

Updated by Karsten Dambekalns almost 11 years ago

  • Status changed from Needs Feedback to On Hold
  • Start date deleted (2010-02-24)

You need to sign the CLA before we can accept your contribution, see http://typo3.org/about/licenses

#5

Updated by Lukas Lentner over 10 years ago

I did

#6

Updated by Christian Müller over 9 years ago

  • Status changed from On Hold to Rejected
  • Has patch set to No

In the meantime the method has changed quite a lot and the limit IMHO makes sense otherwise the output will totally break out. It would probably make sense to log smaller chunks of data if you hit this limit. Reopen if you have a good use-case to change the limit.

Also available in: Atom PDF