logo       
Google Custom Search
    AddThis Social Bookmark Button
-->

[mb-commits] r7914 - mb_server/trunk/htdocs/comp/release_editor: msg#00138

Subject: [mb-commits] r7914 - mb_server/trunk/htdocs/comp/release_editor
Author: keschte
Date: 2006-06-24 14:44:00 +0000 (Sat, 24 Jun 2006)
New Revision: 7914

Modified:
   mb_server/trunk/htdocs/comp/release_editor/entermoderations-add
   mb_server/trunk/htdocs/comp/release_editor/form
   mb_server/trunk/htdocs/comp/release_editor/review
Log:
fixed tickets #1582, #1639


Modified: mb_server/trunk/htdocs/comp/release_editor/entermoderations-add
===================================================================
--- mb_server/trunk/htdocs/comp/release_editor/entermoderations-add     
2006-06-24 12:12:53 UTC (rev 7913)
+++ mb_server/trunk/htdocs/comp/release_editor/entermoderations-add     
2006-06-24 14:44:00 UTC (rev 7914)
@@ -118,7 +118,41 @@
                }
                my $new = "";
 
-               # build release sql.
+               # retrieve 1..n tracks from the arguments
+               my $trackdata = "";
+               my $realtracks = 0;
+               for (my $i = 0; $i < $tracks; $i++)
+               {
+                       # skip deleted tracks, and use internal counter ($i) to
+                       # handle track offsets.
+                       if (!$ARGS{"trackdel$i"})
+                       {
+                               MusicBrainz::TrimInPlace($ARGS{"track$i"});
+                               $trackdata .= "Track" . ($i + 1) . "=" . 
$ARGS{"track$i"} . "\n";
+                               if ($toc ne "")
+                               {
+                                       my $dur = int((($offsets[$i+1] - 
$offsets[$i])*1000)/75);
+                                       $trackdata .= "TrackDur" . ($i + 1) . 
"=" . $dur . "\n";
+                               }
+                               else
+                               {
+                                       my $length = 
Track::UnformatTrackLength($ARGS{"tracklength$i"});
+                                       $length = 0 if $length < 0;
+                                       $trackdata .= "TrackDur" . ($i + 1) . 
"=" . $length . "\n";
+                               }
+                               if ($hasmultipletrackartists)
+                               {
+                                       my $new_artistid = $ARGS{sprintf 
"tr%d_artistid", $i};
+                                       $trackdata .= "ArtistID" . ($i + 1) . 
"=" . $new_artistid . "\n";
+                               }
+                               $realtracks++;
+                       }
+               }
+               # update $tracks variable with real number of tracks
+               # ($tracks-number of deleted tracks)
+               $tracks = $realtracks;
+
+               # build release data.
                if (!defined $artistid)
                {
                        MusicBrainz::TrimInPlace($artistname, $sortname);
@@ -132,31 +166,8 @@
                $new .= "TOC=$toc\n" if ($toc ne "");
                $new .= "FreedbId=$freedbid\n" if ($freedbid ne "");
                $new .= "FreedbCat=$freedbcat\n" if ($freedbcat ne "");
+               $new .= $trackdata;
 
-
-               # retrieve 1..n tracks from the arguments
-               for (my $i = 0; $i < $tracks; $i++)
-               {
-                       MusicBrainz::TrimInPlace($ARGS{"track$i"});
-                       $new .= "Track" . ($i + 1) . "=" . $ARGS{"track$i"} . 
"\n";
-                       if ($toc ne "")
-                       {
-                               my $dur = int((($offsets[$i+1] - 
$offsets[$i])*1000)/75);
-                               $new .= "TrackDur" . ($i + 1) . "=" . $dur . 
"\n";
-                       }
-                       else
-                       {
-                               my $length = 
Track::UnformatTrackLength($ARGS{"tracklength$i"});
-                               $length = 0 if $length < 0;
-                               $new .= "TrackDur" . ($i + 1) . "=" . $length . 
"\n";
-                       }
-                       if ($hasmultipletrackartists)
-                       {
-                               my $new_artistid = $ARGS{sprintf 
"tr%d_artistid", $i};
-                               $new .= "ArtistID" . ($i + 1) . "=" . 
$new_artistid . "\n";
-                       }
-               }
-
                # retrieve the release-dates and countries from the arguments
                for my $j (sort map { /^rls_year-(\d+)$/ ? ($1) : () } keys 
%ARGS)
                {

Modified: mb_server/trunk/htdocs/comp/release_editor/form
===================================================================
--- mb_server/trunk/htdocs/comp/release_editor/form     2006-06-24 12:12:53 UTC 
(rev 7913)
+++ mb_server/trunk/htdocs/comp/release_editor/form     2006-06-24 14:44:00 UTC 
(rev 7914)
@@ -637,25 +637,8 @@
 %      # GUESS ALL, NEXT
 %      # 
-------------------------------------------------------------------------
                                <tr>
-                                       <td colspan="3">&nbsp;</td>
+                                       <td colspan="5">&nbsp;</td>
                                        <td colspan="2">
-                                               <& /comp/form/buttonsubmit,
-                                                       value => 
$ARGS{"SUBMIT_NEXT"},
-                                                       tabindex => ++$tabindex,
-                                                       validate => 1 &>
-
-                                               &nbsp;&nbsp;&nbsp;
-
-                                               <& /comp/form/buttonsubmit,
-                                                       value => 
$ARGS{"SUBMIT_STARTOVER"},
-                                                       tabindex => 
++$tabindex, &>
-
-                                               <& /comp/form/buttonsubmit,
-                                                       value => 
$ARGS{"SUBMIT_CANCEL"},
-                                                       tabindex => 
++$tabindex, &>
-
-                                       </td>
-                                       <td colspan="2">
                                                <& /comp/form/jsbutton, id => 
"BTN_ALL" &>
                                        </td>
                                </tr>
@@ -735,6 +718,17 @@
                                                        value => 
$ARGS{"SUBMIT_NEXT"},
                                                        tabindex => ++$tabindex,
                                                        validate => 1 &>
+
+                                               &nbsp;&nbsp;&nbsp;
+
+                                               <& /comp/form/buttonsubmit,
+                                                       value => 
$ARGS{"SUBMIT_STARTOVER"},
+                                                       tabindex => 
++$tabindex, &>
+
+                                               <& /comp/form/buttonsubmit,
+                                                       value => 
$ARGS{"SUBMIT_CANCEL"},
+                                                       tabindex => 
++$tabindex, &>
+
                                        </td>
                                </tr>
                        </table>

Modified: mb_server/trunk/htdocs/comp/release_editor/review
===================================================================
--- mb_server/trunk/htdocs/comp/release_editor/review   2006-06-24 12:12:53 UTC 
(rev 7913)
+++ mb_server/trunk/htdocs/comp/release_editor/review   2006-06-24 14:44:00 UTC 
(rev 7914)
@@ -133,6 +133,10 @@
                push @relorig, $rel;
        }
 
+       # if editmode is true, we're editing a release, else
+       # we're adding one.
+       my $editmode = $ARGS{"v::action"} ne "/cdi/enter.html";
+
        # prepare 1..n track objects
        my @deletedtracks;
        my @tracks;
@@ -226,6 +230,7 @@
                                        <td class="autoedit"><& .isAutoEdit, 
&ModDefs::MOD_ADD_ALBUM &></td>
                                </tr>
 
+%                              $releasechanges++;
 %                      }
 %                      else
 %                      {


<Prev in Thread] Current Thread [Next in Thread>