More Refactoring, Fixing Bugs (3/18)

Today we have made more improvements to refactoring. It's funny, we made so many code changes, but no real feature was changed. However, we did fix some bugs.

The first bug we fixed was audio not saving after load. We completely removed our timeupdate event function, which allowed us to recode it into a simple 4-line function which saves our audio data. We then added a .position variable to the Chapter class, and allowed the app to save to this variable when the timeupdate event was fired. You can see Alex's explanation of this here.

	$("#audioSource").on("timeupdate", function(e){
	var floatSeconds = $("#audioSource").prop('currentTime'); // change to event
	appUIState.currentChapter.position = floatSeconds;
	console.log("new time " + appUIState.currentChapter.position);
	});

view raw gistfile1.js hosted with ❤ by GitHub

Another bug that was fixed today was the descriptions in search not being formatted properly. This bug was also a simple fix, I had run into it earlier and I showed Alex the code from previously. We determined that it was not parsing the HTML properly. [See line "this.description"]

	function Book(args) {
	this.chapters = args.chapters
	var json = args.json;
	this.json = json;
	this.description = $($.parseHTML(json.description)).text();
	this.title = json.title;
	this.id = json.id;
	}

view raw gistfile1.js hosted with ❤ by GitHub

All that was required was JQuery's parseHTML method, followed by a text() method. This allowed for the app to remove all HTML formatting from the description, which had resulted in weird, random spaces, and bolded/hyperlinked text that we didn't want.

That was about it for today! Refactoring can be time consuming, but in the end it's making it a lot easier to add things on.

EDIT: After further testing, it seems as if we're no longer getting "not well formed" errors, and the page change bug [you had to restart the app to search for a different book] is no longer there!