Show More
The requested changes are too big and content was truncated. Show full diff
1 | NO CONTENT: new file 100644 |
|
NO CONTENT: new file 100644 | ||
The requested commit or file is too big and content was truncated. Show full diff |
@@ -0,0 +1,63 b'' | |||||
|
1 | import logging | |||
|
2 | ||||
|
3 | from sqlalchemy import * | |||
|
4 | from rhodecode.model import meta | |||
|
5 | from rhodecode.lib.dbmigrate.versions import _reset_base, notify | |||
|
6 | ||||
|
7 | log = logging.getLogger(__name__) | |||
|
8 | ||||
|
9 | ||||
|
10 | def get_by_key(cls, key): | |||
|
11 | return cls.query().filter(cls.ui_key == key).scalar() | |||
|
12 | ||||
|
13 | ||||
|
14 | def create_or_update_hook(cls, key, val, SESSION): | |||
|
15 | new_ui = get_by_key(cls, key) or cls() | |||
|
16 | new_ui.ui_section = 'hooks' | |||
|
17 | new_ui.ui_active = True | |||
|
18 | new_ui.ui_key = key | |||
|
19 | new_ui.ui_value = val | |||
|
20 | ||||
|
21 | SESSION().add(new_ui) | |||
|
22 | ||||
|
23 | ||||
|
24 | def upgrade(migrate_engine): | |||
|
25 | """ | |||
|
26 | Upgrade operations go here. | |||
|
27 | Don't create your own engine; bind migrate_engine to your metadata | |||
|
28 | """ | |||
|
29 | _reset_base(migrate_engine) | |||
|
30 | from rhodecode.lib.dbmigrate.schema import db_4_7_0_0 as db | |||
|
31 | ||||
|
32 | # issue fixups | |||
|
33 | fixups(db, meta.Session) | |||
|
34 | ||||
|
35 | ||||
|
36 | def downgrade(migrate_engine): | |||
|
37 | meta = MetaData() | |||
|
38 | meta.bind = migrate_engine | |||
|
39 | ||||
|
40 | ||||
|
41 | def fixups(models, _SESSION): | |||
|
42 | ||||
|
43 | cleanup_if_present = ( | |||
|
44 | models.RhodeCodeUi.HOOK_PRETX_PUSH, | |||
|
45 | ) | |||
|
46 | ||||
|
47 | for hook in cleanup_if_present: | |||
|
48 | ui_cfg = models.RhodeCodeUi.query().filter( | |||
|
49 | models.RhodeCodeUi.ui_key == hook).scalar() | |||
|
50 | if ui_cfg is not None: | |||
|
51 | log.info('Removing RhodeCodeUI for hook "%s".', hook) | |||
|
52 | _SESSION().delete(ui_cfg) | |||
|
53 | ||||
|
54 | to_add = [ | |||
|
55 | (models.RhodeCodeUi.HOOK_PRETX_PUSH, | |||
|
56 | 'python:vcsserver.hooks.pre_push'), | |||
|
57 | ] | |||
|
58 | ||||
|
59 | for hook, value in to_add: | |||
|
60 | log.info('Adding RhodeCodeUI for hook "%s".', hook) | |||
|
61 | create_or_update_hook(models.RhodeCodeUi, hook, value, _SESSION) | |||
|
62 | ||||
|
63 | _SESSION().commit() No newline at end of file |
@@ -51,7 +51,7 b' PYRAMID_SETTINGS = {}' | |||||
51 | EXTENSIONS = {} |
|
51 | EXTENSIONS = {} | |
52 |
|
52 | |||
53 | __version__ = ('.'.join((str(each) for each in VERSION[:3]))) |
|
53 | __version__ = ('.'.join((str(each) for each in VERSION[:3]))) | |
54 |
__dbversion__ = 6 |
|
54 | __dbversion__ = 65 # defines current db version for migrations | |
55 | __platform__ = platform.system() |
|
55 | __platform__ = platform.system() | |
56 | __license__ = 'AGPLv3, and Commercial License' |
|
56 | __license__ = 'AGPLv3, and Commercial License' | |
57 | __author__ = 'RhodeCode GmbH' |
|
57 | __author__ = 'RhodeCode GmbH' |
@@ -315,6 +315,7 b' class DbManage(object):' | |||||
315 | (RhodeCodeUi.HOOK_PRE_PULL, 'python:vcsserver.hooks.pre_pull'), |
|
315 | (RhodeCodeUi.HOOK_PRE_PULL, 'python:vcsserver.hooks.pre_pull'), | |
316 | (RhodeCodeUi.HOOK_PULL, 'python:vcsserver.hooks.log_pull_action'), |
|
316 | (RhodeCodeUi.HOOK_PULL, 'python:vcsserver.hooks.log_pull_action'), | |
317 | (RhodeCodeUi.HOOK_PRE_PUSH, 'python:vcsserver.hooks.pre_push'), |
|
317 | (RhodeCodeUi.HOOK_PRE_PUSH, 'python:vcsserver.hooks.pre_push'), | |
|
318 | (RhodeCodeUi.HOOK_PRETX_PUSH, 'python:vcsserver.hooks.pre_push'), | |||
318 | (RhodeCodeUi.HOOK_PUSH, 'python:vcsserver.hooks.log_push_action'), |
|
319 | (RhodeCodeUi.HOOK_PUSH, 'python:vcsserver.hooks.log_push_action'), | |
319 |
|
320 | |||
320 | ] |
|
321 | ] |
@@ -411,6 +411,7 b' def config_data_from_db(clear_session=Tr' | |||||
411 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRE_PULL)) |
|
411 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRE_PULL)) | |
412 | if 'push' not in enabled_hook_classes: |
|
412 | if 'push' not in enabled_hook_classes: | |
413 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRE_PUSH)) |
|
413 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRE_PUSH)) | |
|
414 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRETX_PUSH)) | |||
414 |
|
415 | |||
415 | config = [entry for entry in config if entry[:2] not in skip_entries] |
|
416 | config = [entry for entry in config if entry[:2] not in skip_entries] | |
416 |
|
417 |
@@ -350,6 +350,7 b' class RhodeCodeUi(Base, BaseModel):' | |||||
350 | HOOK_PRE_PULL = 'preoutgoing.pre_pull' |
|
350 | HOOK_PRE_PULL = 'preoutgoing.pre_pull' | |
351 | HOOK_PULL = 'outgoing.pull_logger' |
|
351 | HOOK_PULL = 'outgoing.pull_logger' | |
352 | HOOK_PRE_PUSH = 'prechangegroup.pre_push' |
|
352 | HOOK_PRE_PUSH = 'prechangegroup.pre_push' | |
|
353 | HOOK_PRETX_PUSH = 'pretxnchangegroup.pre_push' | |||
353 | HOOK_PUSH = 'changegroup.push_logger' |
|
354 | HOOK_PUSH = 'changegroup.push_logger' | |
354 |
|
355 | |||
355 | # TODO: johbo: Unify way how hooks are configured for git and hg, |
|
356 | # TODO: johbo: Unify way how hooks are configured for git and hg, |
@@ -50,8 +50,8 b' class SettingNotFound(Exception):' | |||||
50 | class SettingsModel(BaseModel): |
|
50 | class SettingsModel(BaseModel): | |
51 | BUILTIN_HOOKS = ( |
|
51 | BUILTIN_HOOKS = ( | |
52 | RhodeCodeUi.HOOK_REPO_SIZE, RhodeCodeUi.HOOK_PUSH, |
|
52 | RhodeCodeUi.HOOK_REPO_SIZE, RhodeCodeUi.HOOK_PUSH, | |
53 |
RhodeCodeUi.HOOK_PRE_PUSH, RhodeCodeUi.HOOK_P |
|
53 | RhodeCodeUi.HOOK_PRE_PUSH, RhodeCodeUi.HOOK_PRETX_PUSH, | |
54 | RhodeCodeUi.HOOK_PRE_PULL) |
|
54 | RhodeCodeUi.HOOK_PULL, RhodeCodeUi.HOOK_PRE_PULL) | |
55 | HOOKS_SECTION = 'hooks' |
|
55 | HOOKS_SECTION = 'hooks' | |
56 |
|
56 | |||
57 | def __init__(self, sa=None, repo=None): |
|
57 | def __init__(self, sa=None, repo=None): |
@@ -81,23 +81,27 b' def disable_hooks(request, hooks):' | |||||
81 |
|
81 | |||
82 |
|
82 | |||
83 | HOOK_PRE_PUSH = db.RhodeCodeUi.HOOK_PRE_PUSH |
|
83 | HOOK_PRE_PUSH = db.RhodeCodeUi.HOOK_PRE_PUSH | |
|
84 | HOOK_PRETX_PUSH = db.RhodeCodeUi.HOOK_PRETX_PUSH | |||
84 | HOOK_PUSH = db.RhodeCodeUi.HOOK_PUSH |
|
85 | HOOK_PUSH = db.RhodeCodeUi.HOOK_PUSH | |
85 | HOOK_PRE_PULL = db.RhodeCodeUi.HOOK_PRE_PULL |
|
86 | HOOK_PRE_PULL = db.RhodeCodeUi.HOOK_PRE_PULL | |
86 | HOOK_PULL = db.RhodeCodeUi.HOOK_PULL |
|
87 | HOOK_PULL = db.RhodeCodeUi.HOOK_PULL | |
87 | HOOK_REPO_SIZE = db.RhodeCodeUi.HOOK_REPO_SIZE |
|
88 | HOOK_REPO_SIZE = db.RhodeCodeUi.HOOK_REPO_SIZE | |
88 |
|
89 | |||
89 | HG_HOOKS = frozenset( |
|
90 | HG_HOOKS = frozenset( | |
90 |
(HOOK_PRE_PULL, HOOK_PULL, HOOK_PRE_PUSH, HOOK_PUSH, |
|
91 | (HOOK_PRE_PULL, HOOK_PULL, HOOK_PRE_PUSH, HOOK_PRETX_PUSH, HOOK_PUSH, | |
|
92 | HOOK_REPO_SIZE)) | |||
91 |
|
93 | |||
92 |
|
94 | |||
93 | @pytest.mark.parametrize('disabled_hooks,expected_hooks', [ |
|
95 | @pytest.mark.parametrize('disabled_hooks,expected_hooks', [ | |
94 | ([], HG_HOOKS), |
|
96 | ([], HG_HOOKS), | |
95 | ([HOOK_PRE_PUSH, HOOK_REPO_SIZE], [HOOK_PRE_PULL, HOOK_PULL, HOOK_PUSH]), |
|
|||
96 | (HG_HOOKS, []), |
|
97 | (HG_HOOKS, []), | |
|
98 | ||||
|
99 | ([HOOK_PRE_PUSH, HOOK_PRETX_PUSH, HOOK_REPO_SIZE], [HOOK_PRE_PULL, HOOK_PULL, HOOK_PUSH]), | |||
|
100 | ||||
97 | # When a pull/push hook is disabled, its pre-pull/push counterpart should |
|
101 | # When a pull/push hook is disabled, its pre-pull/push counterpart should | |
98 | # be disabled too. |
|
102 | # be disabled too. | |
99 | ([HOOK_PUSH], [HOOK_PRE_PULL, HOOK_PULL, HOOK_REPO_SIZE]), |
|
103 | ([HOOK_PUSH], [HOOK_PRE_PULL, HOOK_PULL, HOOK_REPO_SIZE]), | |
100 | ([HOOK_PULL], [HOOK_PRE_PUSH, HOOK_PUSH, HOOK_REPO_SIZE]), |
|
104 | ([HOOK_PULL], [HOOK_PRE_PUSH, HOOK_PRETX_PUSH, HOOK_PUSH, HOOK_REPO_SIZE]), | |
101 | ]) |
|
105 | ]) | |
102 | def test_make_db_config_hg_hooks(pylonsapp, request, disabled_hooks, |
|
106 | def test_make_db_config_hg_hooks(pylonsapp, request, disabled_hooks, | |
103 | expected_hooks): |
|
107 | expected_hooks): |
General Comments 0
You need to be logged in to leave comments.
Login now