- Added some comments with tax law basis (#182)

Author: jczaja

src/pl.rs

@@ -56,6 +56,10 @@ fn is_non_working_day(date: &chrono::NaiveDate) -> Result<bool, String> {
         .map_err(|_| format!("Error checking if given date: {date} is holiday"))
 }
 
+// We search a exchange rate from a working day preceeding given date (settlement date for
+// etrade)
+//Art. 11a ust. 1 (Dz.U. 2024 poz. 226):
+// Przychody w walutach obcych przelicza się na złote według kursu średniego walut obcych ogłaszanego przez Narodowy Bank Polski z ostatniego dnia roboczego poprzedzającego dzień uzyskania przychodu.
 // Iterate through dates and find where value is None
 // and then try to get for that specific date from cache
 fn get_exchange_rates_from_cache(
@@ -81,6 +85,7 @@ fn get_exchange_rates_from_cache(
         let mut converted_date = chrono::NaiveDate::parse_from_str(&date, "%m/%d/%y")
             .map_err(|_| format!("Error parsing date: {date}"))?;
         let mut is_working_day = false;
+
         while is_working_day == false {
             converted_date = converted_date
                 .checked_sub_signed(chrono::Duration::days(1))
@@ -135,6 +140,10 @@ fn get_exchange_rates_from_cache(
 }
 
 impl etradeTaxReturnHelper::Residency for PL {
+    // We search a exchange rate from a working day preceeding given date (settlement date for
+    // etrade)
+    //Art. 11a ust. 1 (Dz.U. 2024 poz. 226):
+    // Przychody w walutach obcych przelicza się na złote według kursu średniego walut obcych ogłaszanego przez Narodowy Bank Polski z ostatniego dnia roboczego poprzedzającego dzień uzyskania przychodu.
     fn get_exchange_rates(
         &self,
         dates: &mut std::collections::HashMap<

src/transactions.rs

@@ -99,7 +99,13 @@ pub fn verify_transactions<T>(
 /// Trade date is when transaction was trigerred.
 /// fees and commission are applied at the moment of settlement date so
 /// we ignore those and use net income rather than principal
-/// Actual Tax is to be paid from settlement_date
+/// Actual Tax duty is to be paid using settlement_date
+/// Basis: Art. 17 ust. 1ab pkt 1 PIT
+/// 1ab.  Przychód określony w ust. 1 pkt 6:
+/// 1) z odpłatnego zbycia udziałów (akcji), udziałów w spółdzielni oraz papierów wartościowych powstaje w momencie przeniesienia na nabywcę własności udziałów (akcji), udziałów w spółdzielni oraz papierów wartościowych;
+/// (jczaja): I have spoken with etrade support and they told me that ownership of stocks is
+/// transfered on settlement date      
+
 pub fn reconstruct_sold_transactions(
     sold_transactions: &Vec<(String, String, f32, f32, f32, Option<String>)>,
     gains_and_losses: &Vec<(String, String, f32, f32, f32)>,