Project

General

Profile

Actions

User story #539

closed

Refactor: nom, package, utilisation de méthodes communes

Added by François ARMAND over 13 years ago. Updated over 8 years ago.

Status:
Rejected
Priority:
5
Assignee:
-
Category:
Architecture - Code maintenance
Target version:
-
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

Une liste des points à refactorer dans le prochain run. A completer au fur et à mesure.

Packages/Classes =================

- passer les domain/WB* dans rudder-core
- enlever les repositories qui ne sont plus utilisés dans com.normation.rudder.repository
- merger les PolicyInstanceTarget core et web

Noms à changer ==============

- PolicyPackage => PolicyTempate
- PolicyPackageCategory => PolicyTemplateCategory

- WBServerGroup -> WBNodeGroup

- homogénéiser Name / description / shortDescription / longDescription,
les attributs LDAP correspondants,
leur type (Name:String, shortDescription:String, longDescription:XML ? Des options ou pas ? Valeur par défaut ? )
et leur sémantique

- attributs LDAP : tous les préfixer par "rudder", ou aucun.

- rudderWPI -> rudderPolicyInstance

- rudderServer > rudderNodeConfiguration
trouver un nom pour les "policy instance" et "target policy instance" en dessus de rudder (NodeConfigurationInstance ? Peut-être pas la peine, puisque ca devrait être changé)

- attributs LDAP : voir leur taille, certains (utilisés comme RDN) font 25 chars.

- dans les repository, éviter les redondances telles que: policyInstanceRepository.savePolicyInstance et préférer policyInstanceRepository.save

- trouver un autre nom pour les "CFclerk Policy Instances".

Consistence / id & DN =====================

Dans les Policy*, en LDAP, utiliser des DN à la place des Id (ex: configurationRule#policyInstanceUuid => utiliser un DN)

Actions #1

Updated by Nicolas CHARLES over 13 years ago

Dans le code, on a toujours A_SERVER_UUID qui devrait etre renommé

Je me demande si l'objet Node ne devrait pas etre remonté dans inventory-core

Actions #2

Updated by Nicolas CHARLES over 13 years ago

  • % Done changed from 0 to 10
Actions #3

Updated by François ARMAND over 13 years ago

  • Category set to 13
Actions #4

Updated by François ARMAND almost 13 years ago

Je remet la liste ici, on pourra barre au fur et à mesure ce qui est fait.

Packages/Classes =================

- passer les domain/WB* dans rudder-core
- enlever les repositories qui ne sont plus utilisés dans com.normation.rudder.repository
- merger les PolicyInstanceTarget core et web

Noms à changer ==============

- PolicyPackage => PolicyTempate
- PolicyPackageCategory => PolicyTemplateCategory

- WBServerGroup -> WBNodeGroup (en fait, HS: on n'a plus que NodeGroup)

- homogénéiser Name / description / shortDescription / longDescription,
les attributs LDAP correspondants,
leur type (Name:String, shortDescription:String, longDescription:XML ? Des options ou pas ? Valeur par défaut ? )
et leur sémantique

- attributs LDAP : tous les préfixer par "rudder", ou aucun.

- rudderWPI -> rudderPolicyInstance

- rudderServer > rudderNodeConfiguration

- attributs LDAP : voir leur taille, certains (utilisés comme RDN) font 25 chars.

- dans les repository, éviter les redondances telles que: policyInstanceRepository.savePolicyInstance et préférer policyInstanceRepository.save

- trouver un autre nom pour les "CFclerk Policy Instances".

Consistence / id & DN =====================

Dans les Policy*, en LDAP, utiliser des DN à la place des Id (ex: configurationRule#policyInstanceUuid => utiliser un DN) En fait, on fait plutôt le contraire en décorellant un max le domain de LDAP.

Actions #5

Updated by Jonathan CLARKE almost 13 years ago

  • Category changed from 13 to Architecture - Code maintenance
Actions #6

Updated by François ARMAND over 12 years ago

  • Target version set to 18
Actions #8

Updated by François ARMAND over 12 years ago

  • Target version changed from 18 to 24
Actions #9

Updated by François ARMAND almost 12 years ago

  • Status changed from New to Rejected

That was actually done in 2.4 when we switch from PT/PI/CR to Technique/Directive/Rule

Actions #10

Updated by Jonathan CLARKE almost 12 years ago

  • Target version changed from 24 to 18
Actions #11

Updated by Jonathan CLARKE over 8 years ago

  • Target version deleted (18)
Actions

Also available in: Atom PDF