Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is global_ignore_columns not supported anymore? #81

Closed
k0pernikus opened this issue Nov 4, 2014 · 2 comments
Closed

Is global_ignore_columns not supported anymore? #81

k0pernikus opened this issue Nov 4, 2014 · 2 comments

Comments

@k0pernikus
Copy link

I was working my way through the doc and stumbled across:

  [Symfony\Component\Config\Definition\Exception\InvalidConfigurationException]    
  Unrecognized options "global_ignore_columns" under "simple_things_entity_audit"

And true enough, the sample config does not show "global_ignore_columns".

./app/console config:dump-reference simple_things_entity_audit
# Default configuration for extension with alias: "simple_things_entity_audit"
simple_things_entity_audit:
    audited_entities:     []
    table_prefix:         ''
    table_suffix:         _audit
    revision_field_name:  rev
    revision_type_field_name:  revtype
    revision_table_name:  revisions
    revision_id_field_type:  integer
    listener:
        current_username:     true

So this will run:

simple_things_entity_audit:
    audited_entities:
        - Hn\TrainingBundle\Entity\Trainee
#    global_ignore_columns:
#        - created_at
#        - updated_at
    table_prefix:         ''
    table_suffix:         _audit
    revision_field_name:  rev
    revision_type_field_name:  revtype
    revision_table_name:  revisions
    revision_id_field_type:  integer
    listener:
        current_username: true

Have I erred somewhere on the way in setting this up, or has the global_ignore_config been removed?

@marphi
Copy link

marphi commented Nov 4, 2014

The 'global_ignore_config' was added in the master branch. But you have installed a latest stable version of this library - 0.6

@k0pernikus
Copy link
Author

@marphi Makes sense and is a nice reminder to always look at the documentation of the actual version one is using. Thank you very much.

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

No branches or pull requests

2 participants