Commit 6f611d48 authored by Iago Gómez Salgado's avatar Iago Gómez Salgado

Fix problems overwritting variables

parent 8dc18fdf
...@@ -4,6 +4,7 @@ var PeopleView = (function() { ...@@ -4,6 +4,7 @@ var PeopleView = (function() {
// Referencia a this que permite acceder a las funciones públicas desde las funciones de jQuery. // Referencia a this que permite acceder a las funciones públicas desde las funciones de jQuery.
var self; var self;
var selfpets;
var formId = 'people-form'; var formId = 'people-form';
var listId = 'people-list'; var listId = 'people-list';
...@@ -239,7 +240,7 @@ var PeopleView = (function() { ...@@ -239,7 +240,7 @@ var PeopleView = (function() {
var addPetListeners = function(pet) { var addPetListeners = function(pet) {
$('#pet-' + pet.id + ' a.editPet').click(function() { $('#pet-' + pet.id + ' a.editPet').click(function() {
petToForm(pet); petToForm(pet);
$('input#btnSubmit').val('Modificar'); $('input#petsBtnSubmit').val('Modificar');
}); });
$('#pet-' + pet.id + ' a.deletePet').click(function() { $('#pet-' + pet.id + ' a.deletePet').click(function() {
...@@ -276,7 +277,7 @@ var PeopleView = (function() { ...@@ -276,7 +277,7 @@ var PeopleView = (function() {
<input name="kind" type="text" value=""/>\ <input name="kind" type="text" value=""/>\
<input name="breed" type="text" value=""/>\ <input name="breed" type="text" value=""/>\
<input name="owner" type="hidden" value=""/>\ <input name="owner" type="hidden" value=""/>\
<input id="btnSubmit" type="submit" value="Create"/>\ <input id="petsBtnSubmit" type="submit" value="Create"/>\
<input id="btnClear" type="reset" value="Limpiar"/>\ <input id="btnClear" type="reset" value="Limpiar"/>\
</form>' </form>'
); );
...@@ -298,6 +299,7 @@ var PeopleView = (function() { ...@@ -298,6 +299,7 @@ var PeopleView = (function() {
} }
var petToRow = function(pet) { var petToRow = function(pet) {
console.log(pet);
return '<tr id="pet-' + pet.id + '">\ return '<tr id="pet-' + pet.id + '">\
<td class="name">' + pet.name + '</td>\ <td class="name">' + pet.name + '</td>\
<td class="kind">' + pet.kind + '</td>\ <td class="kind">' + pet.kind + '</td>\
...@@ -314,6 +316,7 @@ var PeopleView = (function() { ...@@ -314,6 +316,7 @@ var PeopleView = (function() {
function petToForm(pet) { function petToForm(pet) {
var form = $(petsFormQuery); var form = $(petsFormQuery);
console.log(pet.id);
form.find('input[name="petId"]').val(pet.id); form.find('input[name="petId"]').val(pet.id);
form.find('input[name="petName"]').val(pet.name); form.find('input[name="petName"]').val(pet.name);
form.find('input[name="kind"]').val(pet.kind); form.find('input[name="kind"]').val(pet.kind);
...@@ -329,7 +332,7 @@ var PeopleView = (function() { ...@@ -329,7 +332,7 @@ var PeopleView = (function() {
Especie: <input name="kind" type="text" value="" />\ Especie: <input name="kind" type="text" value="" />\
Raza: <input name="breed" type="text" value="" />\ Raza: <input name="breed" type="text" value="" />\
Propietario: <input name="owner" type="text" value="" />\ Propietario: <input name="owner" type="text" value="" />\
<input id="btnSubmit" type="submit" value="Crear"/>\ <input id="petsBtnSubmit" type="submit" value="Crear"/>\
<input id="btnClear" type="reset" value="Limpiar"/>\ <input id="btnClear" type="reset" value="Limpiar"/>\
</form>' </form>'
); );
...@@ -346,35 +349,39 @@ var PeopleView = (function() { ...@@ -346,35 +349,39 @@ var PeopleView = (function() {
}; };
}; };
function PetsView() { function PetsView() {
self = this;
selfpets = this;
this.init = function() { this.init = function() {
petsForm(); petsForm();
$(petsFormQuery).submit(function(event) { $(petsFormQuery).submit(function(event) {
var pet = self.getPetInForm();
console.log(pet); var pet = selfpets.getPetInForm();
if (self.isEditing()) {
if (selfpets.isEditing()) {
pets.modifyPet(pet, pets.modifyPet(pet,
function(pet) { function(pet) {
$('#pet-' + pet.id + ' td.name').text(pet.name); $('#pet-' + pet.id + ' td.name').text(pet.name);
$('#pet-' + pet.id + ' td.kind').text(pet.kind); $('#pet-' + pet.id + ' td.kind').text(pet.kind);
$('#pet-' + pet.id + ' td.breed').text(pet.breed); $('#pet-' + pet.id + ' td.breed').text(pet.breed);
$('#pet-' + pet.id + ' td.owner').text(pet.owner); $('#pet-' + pet.id + ' td.owner').text(pet.owner);
self.resetForm(); selfpets.resetForm();
}, },
showErrorMessage, showErrorMessage,
self.enableForm selfpets.enableForm
); );
} else { } else {
console.log("ola");
pets.addPet(pet, pets.addPet(pet,
function(pet) { function(pet) {
self.resetForm(); selfpets.resetForm();
}, },
showErrorMessage, showErrorMessage,
self.enableForm selfpets.enableForm
); );
} }
...@@ -384,9 +391,8 @@ var PeopleView = (function() { ...@@ -384,9 +391,8 @@ var PeopleView = (function() {
$('#btnClear').click(this.resetForm); $('#btnClear').click(this.resetForm);
}; };
this.getPetInForm = function() { selfpets.getPetInForm = function() {
var form = $(petsFormQuery); var form = $(petsFormQuery);
console.log(form);
return { return {
'id': form.find('input[name="petId"]').val(), 'id': form.find('input[name="petId"]').val(),
'name': form.find('input[name="petName"]').val(), 'name': form.find('input[name="petName"]').val(),
...@@ -396,7 +402,7 @@ var PeopleView = (function() { ...@@ -396,7 +402,7 @@ var PeopleView = (function() {
}; };
}; };
this.getPetInRow = function(id) { selfpets.getPetInRow = function(id) {
var row = $('#pet-' + id); var row = $('#pet-' + id);
if (row !== undefined) { if (row !== undefined) {
...@@ -412,11 +418,9 @@ var PeopleView = (function() { ...@@ -412,11 +418,9 @@ var PeopleView = (function() {
} }
}; };
this.editPet = function(id) { selfpets.editPet = function(id) {
var row = $('#pet-' + id); var row = $('#pet-' + id);
console.log("imprimindo fila:");
console.log(row);
if (row !== undefined) { if (row !== undefined) {
var form = $(petsFormQuery); var form = $(petsFormQuery);
...@@ -428,22 +432,22 @@ var PeopleView = (function() { ...@@ -428,22 +432,22 @@ var PeopleView = (function() {
} }
} }
this.isEditing = function() { selfpets.isEditing = function() {
return $(petsFormQuery + ' input[name="id"]').val() != ""; return $(petsFormQuery + ' input[name="petId"]').val() != "";
}; };
this.disableForm = function() { selfpets.disableForm = function() {
$(petsFormQuery + ' input').prop('disabled', true); $(petsFormQuery + ' input').prop('disabled', true);
}; };
this.enableForm = function() { selfpets.enableForm = function() {
$(petsFormQuery + ' input').prop('disabled', false); $(petsFormQuery + ' input').prop('disabled', false);
}; };
this.resetForm = function() { selfpets.resetForm = function() {
$(petsFormQuery)[0].reset(); $(petsFormQuery)[0].reset();
$(petsFormQuery + ' input[name="id"]').val(''); $(petsFormQuery + ' input[name="petId"]').val('');
$('#btnSubmit').val('Crear'); $('#petsBtnSubmit').val('Crear');
}; };
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment