Skip to content

Instantly share code, notes, and snippets.

@cburgdorf
Created March 4, 2011 17:49
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save cburgdorf/855371 to your computer and use it in GitHub Desktop.
Save cburgdorf/855371 to your computer and use it in GitHub Desktop.
//Consider this as part of a bigger classlike object
.bind('NewButtonClicked.BaseEditWidget.FahrerController', function (event, data) {
if (isInEditMode()) {
dialogWidget.question('Änderungen verwerfen?', 'Sie befinden sich im Bearbeitungsmodus. Möchten Sie den Bearbeitungsmodus beenden und ggf. getätigte Änderungen verwerfen?').done(function () {
//This code is duplicate because I dont want to clutter my class like object with too many
//tiny trivial methods. However, wrapping the snippet in a function just inside this handler
//should be fine or not? (Look at the refactoring at the bottom)
currentEntity = createNewEntity();
fahrerEditWidget.loadFahrer(currentEntity);
fahrerEditWidget.enable();
});
} else {
currentEntity = createNewEntity();
fahrerEditWidget.loadFahrer(currentEntity);
fahrerEditWidget.enable();
}
});
//Refactoring
.bind('NewButtonClicked.BaseEditWidget.FahrerController', function (event, data) {
var createNewAndLoad = function () {
currentEntity = createNewEntity();
fahrerEditWidget.loadFahrer(currentEntity);
fahrerEditWidget.enable();
};
if (isInEditMode()) {
dialogWidget.question('Änderungen verwerfen?', 'Sie befinden sich im Bearbeitungsmodus. Möchten Sie den Bearbeitungsmodus beenden und ggf. getätigte Änderungen verwerfen?').done(function () {
createNewAndLoad();
});
} else {
createNewAndLoad();
}
});
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment