[ckan-changes] [ckan/ckan] 867f3f: changed parameters to wrapper() funcs in decorator...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 867f3fae92f0d98c99e231d1070b20dd5c490dfe
      https://github.com/ckan/ckan/commit/867f3fae92f0d98c99e231d1070b20dd5c490dfe
  Author: Alice Butcher <alycejenni at gmail.com>
  Date:   2018-02-28 (Wed, 28 Feb 2018)

  Changed paths:
    M ckan/logic/__init__.py

  Log Message:
  -----------
  changed parameters to wrapper() funcs in decorators to args/kwargs

wrapper functions do not actually use the parameters so they don't need to be defined; defining them was causing issues with chained functions as there are 3 parameters passed to the wrapper


  Commit: 352457ccf25e428d22ba2eb9997e4a0880f3e9a7
      https://github.com/ckan/ckan/commit/352457ccf25e428d22ba2eb9997e4a0880f3e9a7
  Author: Alice Butcher <alycejenni at gmail.com>
  Date:   2018-03-01 (Thu, 01 Mar 2018)

  Changed paths:
    M ckan/logic/__init__.py

  Log Message:
  -----------
  set attribute directly on function instead of using internal def

as the decorators are only setting simple attributes, the internal wrapper() def block is unnecessary.


  Commit: 400f17458217424b130d90577edef763316a2c80
      https://github.com/ckan/ckan/commit/400f17458217424b130d90577edef763316a2c80
  Author: Ian Ward <ian at excess.org>
  Date:   2018-03-07 (Wed, 07 Mar 2018)

  Changed paths:
    M ckan/logic/__init__.py

  Log Message:
  -----------
  Merge pull request #4048 from alycejenni/wrapper-generic-params

Make parameters for wrapper() functions in ckan.logic generic args/kwargs


Compare: https://github.com/ckan/ckan/compare/c74fc2608136...400f17458217