search

hqchai / UCLA-Translate-Flashcards

0 stars 0 forks source link

Deck.html (and maybe others) are linked to "home.jsp" #13

Closed keylime503 closed 10 years ago

keylime503 commented 10 years ago

I noticed deck.html links to home.jsp instead of /home. Might be some other html pages that we added today that do the same and need to be fixed.

brookewenig commented 10 years ago

That's because that's deck.html - we should use deck.jsp, but it has null ptr error because Flashcard class/flashcardList isn't implemented.

jisaji commented 10 years ago

Delete deck.HTML then. If a file or code isn't being used currently then it shouldn't be in the repository. If we need it later we can look it up in a previous commit.

I think home.jsp also suffers from a similar old code problem.

keylime503 commented 10 years ago

Regardless, since home.jsp is called by the HomeServlet, home.jsp should NEVER be called directly. Instead, it should be "/home"