paoda
/
melodii
Archived
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Comment written about serious bug 

Unable to replicate atm so no immediate fix incoming.
Keeping notice for future reference.
This commit is contained in:
Paoda 2019-02-13 16:09:47 -06:00
parent b413193cb7
commit 463cbf0bc8
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/components/Body.js
View File

@ -29,10 +29,12 @@ export default class Body extends React.Component {
this.initialize(); this.initialize();
} }
shouldComponentUpdate(nextProps, state) { shouldComponentUpdate(nextProps, state) {
return this.state.table.id !== state.table.id; if (this.state.table.id) return this.state.table.id !== state.table.id;
// On Feb 13 2019 Had problem wehre this.state.table.id was undefined.
// Unable to replicate issue, but it's a serous one. Probably should fix this yeah?
} }
render() { render() {
console.warn("Render:", this.state.table); console.log("Render TableID: " + this.state.table.id);
return ( return (
<div className="wrapper"> <div className="wrapper">
<div id="searchBar"> <div id="searchBar">

2
src/melodii/MusicPlayer.js
View File

@ -52,7 +52,7 @@ export default class MusicPlayer {
try { try {
this.element.src = this.getURICompatible(path); this.element.src = this.getURICompatible(path);
this.element.load(); this.element.load();
console.log(path + " succesfully loaded"); console.log(path + " was succesfully loaded");
} catch (e) { } catch (e) {
console.error(path + " failed to load: " + e.name); console.error(path + " failed to load: " + e.name);
} }