View Issue Details

IDProjectCategoryView StatusLast Update
0001708AJAX/JSBug Report - Interfacepublic2009-10-20 21:39
Reporterpermidion Assigned Tofahrenheit  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Summary0001708: massaddep fail when adding a new non-standart ep
DescriptionI tried to add a new ep of type opening/ending in Saki, so I click on the [+] after the last C ep (C5) in the list, a new line is displayed, but the ep id is set to "6" instead of "C6", which cause a fatal error "The server encountered an internal error or misconfiguration and was unable to complete your request."

Doing the same but with manually modifying the ep id to "C6" before submitting works ok.
TagsAJAX, massep

Activities

Shitenshi

2009-10-11 04:39

reporter   ~0003011

I'm using AniDB without AJAX, and don't seem to have this problem. Adding a new episode after the last Cx (for example C6), will automatically generate a C7 episode correctly.

noec

2009-10-17 16:21

developer   ~0003019

The issue seems to be because of the difference how createNewEpisode(epType,epNo) and CAnimeEntry(node) handles episode type.

createNewEpisode(epType,epNo) (anime3/addemp.js; line 168):
it sets episode type to the "flag" node.

CAnimeEntry(node) (anime3/parser.js; line 179):
it tries to find episode type in "epno" node.

Not making bugfix myself, not to brake anything. ;-)

fahrenheit

2009-10-18 09:35

reporter   ~0003020

was just missing a eptype in the episode epno node to mirror the last change to the xmls, sorry for the delay

nwa

2009-10-20 10:49

administrator   ~0003025

lol
that's so like you, fahrenheit you little rascal.
You fixed the massep add bug by including the corresponding letter, but now broke the numbering.
It's actually quite funky. When using the plus icon to increment the episodes by 1, it adds an extra 1 at the end. So 1 becomes 11, 11 becomes 111 etc.
When using "Add # new episode(s) of type: <type>" it still adds an extra 1.

fahrenheit

2009-10-20 15:12

reporter   ~0003026

crap, it's the issue with untyped languages, i'll fix it tonight, bug me.

Issue History

Date Modified Username Field Change
2009-10-10 17:38 permidion New Issue
2009-10-11 04:39 Shitenshi Note Added: 0003011
2009-10-11 04:39 Shitenshi Tag Attached: massep
2009-10-17 02:12 DerIdiot Project AniDB Website => AJAX/JS
2009-10-17 16:21 noec Note Added: 0003019
2009-10-17 22:39 fahrenheit Status new => assigned
2009-10-17 22:39 fahrenheit Assigned To => fahrenheit
2009-10-18 09:35 fahrenheit Note Added: 0003020
2009-10-18 09:35 fahrenheit Status assigned => resolved
2009-10-18 09:35 fahrenheit Resolution open => fixed
2009-10-20 10:49 nwa Note Added: 0003025
2009-10-20 10:50 nwa Status resolved => assigned
2009-10-20 10:50 nwa Tag Attached: AJAX
2009-10-20 10:50 nwa Resolution fixed => reopened
2009-10-20 15:12 fahrenheit Note Added: 0003026
2009-10-20 21:39 fahrenheit Status assigned => resolved
2009-10-20 21:39 fahrenheit Resolution reopened => fixed