Zbiór zmian 611

Data:

2009-04-27 10:19:42 (17 years ago)

Autor:

marek

Opis:

re #160 - poprawa dodawania klienta

Lokalizacja:

trunk

Pliki:

• BazaReklam.Updater/FormUpdate.cs (zmodyfikowane) (1 diff)
• BazaReklam/AddClient.Designer.cs (zmodyfikowane) (1 diff)
• BazaReklam/AddClient.cs (zmodyfikowane) (1 diff)
• BazaReklam/Baza Reklam.csproj (zmodyfikowane) (1 diff)
• BazaReklam/Classes/Helpers/PdfHelper.cs (zmodyfikowane) (1 diff)
• BazaReklam/Classes/Interfaces/IInvoiceProvider.cs (dodane)
• BazaReklam/Classes/InvoiceProvider.cs (zmodyfikowane) (1 diff)
• BazaReklam/Classes/InvoiceProviderFactory.cs (zmodyfikowane) (3 diffs)
• BazaReklam/Docs/versioninfo.html (zmodyfikowane) (1 diff)
• BazaReklam/FakturowaniePoznanKatowice.cs (zmodyfikowane) (1 diff)
• BazaReklam/Properties/AssemblyInfo.cs (zmodyfikowane) (1 diff)
• BazaReklam/ZamowieniaForm.cs (zmodyfikowane) (11 diffs)
• BazaReklamSetup/BazaReklamSetup.vdproj (zmodyfikowane) (2 diffs)

trunk/BazaReklam.Updater/FormUpdate.cs

@@ -99 +99 @@
             {
                 if (_args.Length < 2)
-                    throw new Exception("B³êdne wywo³anie programu!");
+                    throw new ApplicationException("B³êdne wywo³anie programu!");
 
                 _isBusy = true;

trunk/BazaReklam/AddClient.Designer.cs

@@ -1050 +1050 @@
             this.Text = "Dodaj/modyfikuj klienta";
             this.Load += new System.EventHandler(this.AddClient_Load);
+            this.FormClosing += new System.Windows.Forms.FormClosingEventHandler(this.AddClient_FormClosing);
             this.gbAddress.ResumeLayout(false);
             this.gbAddress.PerformLayout();

trunk/BazaReklam/AddClient.cs

@@ -246 +246 @@
             errorProvider.SetError(txtClientCode, sb.ToString());
         }
+
+        private void AddClient_FormClosing(object sender, FormClosingEventArgs e)
+        {
+            rEKLAMADataSet.KLIENCI.RejectChanges();
+        }
     }
 }

trunk/BazaReklam/Baza Reklam.csproj

@@ -114 +114 @@
     <Compile Include="Classes\Helpers\StringHelper.cs" />
     <Compile Include="Classes\Helpers\WebRequestHelper.cs" />
+    <Compile Include="Classes\Interfaces\IInvoiceProvider.cs" />
     <Compile Include="Classes\Interfaces\IForm.cs" />
     <Compile Include="Classes\Model\Invoice.cs" />

trunk/BazaReklam/Classes/Helpers/PdfHelper.cs

@@ -24 +24 @@
         public bool DisplayPdf()
         {
-            throw new System.Net.WebException("test");
-
-
             string url = ConfigurationHelper.GetSettingByKey("Application.Pdf.Date") ??
              "http://truck.pl/ogloszenia/_getFile.asp?pdfdate,{0},,";

trunk/BazaReklam/Classes/InvoiceProvider.cs

@@ +1 @@
+using Baza_Reklam.Classes.Interfaces;
+
 namespace Baza_Reklam
 {
-    public class InvoiceProvider
+    public class InvoiceProvider : IInvoiceProvider
     {
         private string _city;

trunk/BazaReklam/Classes/InvoiceProviderFactory.cs

@@ -3 +3 @@
 using System.Data;
 using System.Data.SqlClient;
+using Baza_Reklam.Classes.Interfaces;
 
 namespace Baza_Reklam
@@ -9 +10 @@
     {
         private static InvoiceProviderFactory _instance;
-        private readonly List<InvoiceProvider> _invoiceProviders = new List<InvoiceProvider>();
+        private readonly List<IInvoiceProvider> _invoiceProviders = new List<IInvoiceProvider>();
 
         private InvoiceProviderFactory()
@@ -80 +81 @@
         }
 
-        public static List<InvoiceProvider> GetInvoiceProviders()
+        public static List<IInvoiceProvider> GetInvoiceProviders()
         {
             return Instance._invoiceProviders;
         }
 
-        public static InvoiceProvider GetInvoiceProviderById(int invoiceProviderId)
+        public static IInvoiceProvider GetInvoiceProviderById(int invoiceProviderId)
         {
             return
                 Instance._invoiceProviders.Find(
-                    delegate(InvoiceProvider invoiceProvider) { return invoiceProvider.Id == invoiceProviderId; });
+                    delegate(IInvoiceProvider invoiceProvider) { return invoiceProvider.Id == invoiceProviderId; });
         }
 
-        public static List<InvoiceProvider> GetInvoiceProvidersById(int invoiceProviderId)
+        public static List<IInvoiceProvider> GetInvoiceProvidersById(int invoiceProviderId)
         {
             return
                 Instance._invoiceProviders.FindAll(
-                    delegate(InvoiceProvider invoiceProvider) { return invoiceProvider.Id == invoiceProviderId; });
+                    delegate(IInvoiceProvider invoiceProvider) { return invoiceProvider.Id == invoiceProviderId; });
         }
     }

trunk/BazaReklam/Docs/versioninfo.html

@@ -6 +6 @@
 <body>
    <h1>Baza reklam - Informacje o wersji</h1>
+   <div>
+        <a id="1.1.6" />
+        <h2>Wersja 1.1.6 (2009-04-27)</h2>
+        <p>Opis zmian wprowadzonych do wersji 1.1.6</p>
+        <ul>
+            <li>Zgłoszenie #160: Bieżące poprawki na podstawie logu błędów</li>
+            <li>Zgłoszenie #165: Naprawa błędów występujących podczas aktualizacji</li>
+        </ul>
+   </div>
    <div>
         <a id="1.1.5" />

trunk/BazaReklam/FakturowaniePoznanKatowice.cs

@@ -594 +594 @@
             if (pozycjeFakturDlaKlientow.Rows.Count == 0 || naglowkiFakturDlaKlientow.Rows.Count == 0)
             {
-                throw new Exception("Nie ma co fakturowaæ");
+                //TODO: check is there is anythig to invoice before callind that method
+                throw new ApplicationException("Nie ma co fakturowaæ - Poznañ | Katowice");
             }
 

trunk/BazaReklam/Properties/AssemblyInfo.cs

@@ -30 +30 @@
 //
 [assembly: AssemblyVersion("1.0.0.0")]
-[assembly: AssemblyFileVersion("1.1.5")]
+[assembly: AssemblyFileVersion("1.1.6")]

trunk/BazaReklam/ZamowieniaForm.cs

@@ -6 +6 @@
 using System.Windows.Forms;
 using Baza_Reklam.Classes.Helpers;
+using Baza_Reklam.Classes.Interfaces;
 
 namespace Baza_Reklam
@@ -294 +295 @@
             if (zamowieniaBindingSource.Current == null) return;
 
-            if (rEKLAMADataSet.REKLAMA.Rows.Count <= 0)
-            {
-                MessageBox.Show("Proszê dodaæ przynajmniej jedn¹ reklamê do zamówienia.", "Faktura");
-                return;
-            }
-            
-            Cursor = Cursors.WaitCursor;
-
-            DataRowView z = (DataRowView) zamowieniaBindingSource.Current;
-            REKLAMADataSet.zamowieniaRow zamowienie = (REKLAMADataSet.zamowieniaRow) z.Row;
-
-            REKLAMADataSet.KLIENCIRow _klient =
-                (REKLAMADataSet.KLIENCIRow) klienciTableAdapter.GetDataByCustomerId(zamowienie.idKlienta).Rows[0];
-            string errorMessage = czyMoznaZafakturowac(zamowienie, _klient);
-
-            if (errorMessage != string.Empty)
-            {
-                MessageBox.Show(errorMessage);
+            try
+            {
+                Cursor = Cursors.WaitCursor;
+
+                DataRowView z = (DataRowView) zamowieniaBindingSource.Current;
+                REKLAMADataSet.zamowieniaRow zamowienie = (REKLAMADataSet.zamowieniaRow) z.Row;
+
+                REKLAMADataSet.KLIENCIRow klientRow =
+                    (REKLAMADataSet.KLIENCIRow) klienciTableAdapter.GetDataByCustomerId(zamowienie.idKlienta).Rows[0];
+
+                InitIssueTables(zamowienie);
+
+                string errorMessage = CzyMoznaZafakturowac(zamowienie, klientRow);
+
+                if (errorMessage != string.Empty)
+                {
+                    MessageBox.Show(errorMessage, "Fakturowanie");
+                    Cursor = Cursors.Default;
+                    return;
+                }
+
+                WystawFakture(zamowienie, klientRow, false);
+
+                RestoreConnections();
+
+                zamowieniaBindingSource.ResetBindings(false);
+
+                fAKTURYBindingSource.MoveLast();
+            }
+            finally
+            {
                 Cursor = Cursors.Default;
-                return;
-            }
-
-            WystawFakture(zamowienie, _klient, false);
-
-            RestoreConnections();
-
-            zamowieniaBindingSource.ResetBindings(false);
-
-            fAKTURYBindingSource.MoveLast();
-
-            Cursor = Cursors.Default;
+            }
         }
 
@@ -523 +526 @@
         }
 
-        private void PobierzOznaczEmisje(REKLAMADataSet.zamowieniaRow zamowienie, REKLAMADataSet.FAKTURYRow naglowekFaktury)
+
+        private void InitIssueTables(REKLAMADataSet.zamowieniaRow zamowienie)
         {
             emisjeTableAdapter.ClearBeforeFill = false;
             datyWydanTableAdapter.ClearBeforeFill = false;
+            rEKLAMADataSet.UKAZE_SIE_W_NR.Clear();
+            rEKLAMADataSet.DatyWydan.Clear();
 
             REKLAMADataSet.DatyWydanDataTable t = datyWydanTableAdapter.GetDataByPierwszaEmisjaWZamowieniu(zamowienie.idZamowienia);
-
-            if (t.Rows.Count == 0)
-            {
-                //TODO: moze lepiej uzyc MessageBox'a i return niz wywalac wyjatek...?
-                // wyjatki sa dosc kosztowne dla aplikacji
-                throw new Exception("Nie ma co fakturowaæ!");
-            }
+            if (t.Rows.Count <= 0) return;
 
             DateTime d = t[0].DATA_W;
@@ -575 +575 @@
                 reklama.EndEdit();
             }
-
-            if (rEKLAMADataSet.UKAZE_SIE_W_NR.Rows.Count <= 0)
-                throw new ApplicationException("Nie znaleziono ¿adnych emisji do zafakturowania! (" +
-                                               zamowienie.idZamowienia + ": " + StringHelper.ListToString(adIds, ",") + ")");
-
+        }
+
+
+        private void PobierzOznaczEmisje(REKLAMADataSet.zamowieniaRow zamowienie, REKLAMADataSet.FAKTURYRow naglowekFaktury)
+        {
+            //InitIssueTables(zamowienie);
 
             if (zamowienie.IsdataOstatniejZafakturowanejEmisjiNull())
@@ -635 +636 @@
         {
             SLOWNIKDataSet.AGENCIRow agent = (SLOWNIKDataSet.AGENCIRow)agenciTableAdapter.GetDataByKodAgenta(zamowienie.kodAgenta).Rows[0];
-            InvoiceProvider invoiceProvider = InvoiceProviderFactory.GetInvoiceProviderById(agent.InvoiceProviderId);
+            IInvoiceProvider invoiceProvider = InvoiceProviderFactory.GetInvoiceProviderById(agent.InvoiceProviderId);
 
 
@@ -709 +710 @@
                 if (kurs.Count == 0)
                 {
-                    throw new Exception("B³¹d przy pobieraniu kursu waluty.");
+                    throw new ApplicationException("B³¹d przy pobieraniu kursu waluty.");
                 }
 
@@ -730 +731 @@
         
 
-        private string czyMoznaZafakturowac(REKLAMADataSet.zamowieniaRow zamowienie, REKLAMADataSet.KLIENCIRow klientRow)
+        private string CzyMoznaZafakturowac(REKLAMADataSet.zamowieniaRow zamowienie, REKLAMADataSet.KLIENCIRow klientRow)
         {
             string errorMessage = string.Empty;
@@ -738 +739 @@
             errorMessage += zamowienie.IsidKontaNull() ? "Nie wybrano konta. \n" : "";
             errorMessage += zamowienie.IssposobZaplatyNull() ? "Nie wybrano sposobu zap³aty. \n" : "";
-            errorMessage += klientRow.IsCountryNull() || klientRow.Country == string.Empty ? "Brak pañstwa klienta. \n" : "";
-            errorMessage += klientRow.IsAdres_FkaturaNull() || klientRow.Adres_Fkatura == string.Empty ? "Brak adresu faktury klienta. \n" : "";
-            errorMessage += klientRow.IsNipNull() || klientRow.Nip == string.Empty ? "Brak nipu klienta. \n" : "";
+            errorMessage += string.IsNullOrEmpty(klientRow.Country) ? "Brak pañstwa klienta. \n" : "";
+            errorMessage += string.IsNullOrEmpty(klientRow.Adres_Fkatura) ? "Brak adresu faktury klienta. \n" : "";
+            errorMessage +=  string.IsNullOrEmpty(klientRow.Nip) ? "Brak nipu klienta. \n" : "";
 
             if (klientRow.Country != "Polska")
-            {
-                errorMessage += klientRow.IsnipKrajNull() || klientRow.nipKraj == string.Empty ? "Brak symbolu kraju w nipie klienta. \n" : "";
+                errorMessage += string.IsNullOrEmpty(klientRow.nipKraj) ? "Brak symbolu kraju w nipie klienta. \n" : "";
+
+            if (rEKLAMADataSet.REKLAMA.Rows.Count <= 0)
+            {
+                // try to load the ads once again in case something went wrong before...
+                rEKLAMATableAdapter.FillByIdZamowienia(rEKLAMADataSet.REKLAMA, zamowienie.idZamowienia);
+
+                if (rEKLAMADataSet.REKLAMA.Rows.Count <= 0)
+                    errorMessage += "Proszê dodaæ przynajmniej jedn¹ reklamê do zamówienia \n";
             }
 
             if (rEKLAMADataSet.REKLAMA.Select("[ZATWIERDZONO DO DRUKU] = 0").Length > 0)
-            {
                 errorMessage += "W zamówieniu s¹ reklamy niezatwierdzone do druku \n";
-            }
+
+            if (errorMessage.Length > 0)
+                return errorMessage;
+
+            if (rEKLAMADataSet.UKAZE_SIE_W_NR.Rows.Count <= 0)
+            {
+                errorMessage += "Nie znaleziono ¿adnych emisji do zafakturowania \n";
+                return errorMessage;
+            }
+
+            REKLAMADataSet.DatyWydanDataTable t = datyWydanTableAdapter.GetDataByPierwszaEmisjaWZamowieniu(zamowienie.idZamowienia);
+            if (t.Rows.Count == 0)
+                errorMessage += "Nie znaleziono ¿adnych emisji do zafakturowania \n";
 
             return errorMessage;
@@ -1359 +1378 @@
             if (zamowieniaBindingSource.Current == null) return;
 
-            if (rEKLAMADataSet.REKLAMA.Rows.Count <= 0)
-            {
-                MessageBox.Show("Proszê dodaæ przynajmniej jedn¹ reklamê do zamówienia.", "Faktura proforma");
-                return;
-            }
-
             Cursor = Cursors.WaitCursor;
 
@@ -1372 +1385 @@
             REKLAMADataSet.KLIENCIRow _klient =
                 (REKLAMADataSet.KLIENCIRow) klienciTableAdapter.GetDataByCustomerId(zamowienie.idKlienta).Rows[0];
-            string errorMessage = czyMoznaZafakturowac(zamowienie, _klient);
+
+            InitIssueTables(zamowienie);
+
+            string errorMessage = CzyMoznaZafakturowac(zamowienie, _klient);
 
             if (errorMessage != string.Empty)
             {
-                MessageBox.Show(errorMessage);
+                MessageBox.Show(errorMessage, "Fakturowanie - Proforma");
                 Cursor = Cursors.Default;
                 return;
@@ -1391 +1407 @@
             {
                 DataRow[] rows = rEKLAMADataSet.FAKTURY.Select("id_faktury=" + id);
-                usunFakture((REKLAMADataSet.FAKTURYRow) rows[0]);
+                if (rows.Length > 0)
+                    usunFakture((REKLAMADataSet.FAKTURYRow) rows[0]);
+                
                 RestoreConnections();
                 Cursor = Cursors.Default;

trunk/BazaReklamSetup/BazaReklamSetup.vdproj

@@ -725 +725 @@
         "Name" = "8:Microsoft Visual Studio"
         "ProductName" = "8:Baza Reklam"
-        "ProductCode" = "8:{DE903920-0572-4BA7-8623-E07D2868A25C}"
-        "PackageCode" = "8:{0E35FC2A-8DC5-4E25-8EF4-041C314C5E0B}"
+        "ProductCode" = "8:{E94CA729-B311-4DE5-BF21-DF116210CC02}"
+        "PackageCode" = "8:{724D56E2-C502-4EF3-9B3E-C5044DE7D117}"
         "UpgradeCode" = "8:{4E2DBBA4-3139-4790-8DDB-7AADFC963A7D}"
         "RestartWWWService" = "11:FALSE"
@@ -732 +732 @@
         "DetectNewerInstalledVersion" = "11:TRUE"
         "InstallAllUsers" = "11:TRUE"
-        "ProductVersion" = "8:1.1.5"
+        "ProductVersion" = "8:1.1.6"
         "Manufacturer" = "8:AACT"
         "ARPHELPTELEPHONE" = "8:"