Monica-CodingWorld commented on PR #5264:
URL: https://github.com/apache/fineract/pull/5264#issuecomment-3707654250

   > I am afraid my advice was not correct... i see it is still failing in some 
situations.
   > 
   > Let try with this please:
   > 
   > ```
   > Index: 
fineract-provider/src/main/resources/db/changelog/tenant/parts/0071_1_fix_invalid_filename_for_0072.xml
   > IDEA additional info:
   > Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
   > <+>UTF-8
   > ===================================================================
   > diff --git 
a/fineract-provider/src/main/resources/db/changelog/tenant/parts/0071_1_fix_invalid_filename_for_0072.xml
 
b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0071_1_fix_invalid_filename_for_0072.xml
   > --- 
a/fineract-provider/src/main/resources/db/changelog/tenant/parts/0071_1_fix_invalid_filename_for_0072.xml
      (revision 6187378e83873aaf3fe911f00f9ea085460dd795)
   > +++ 
b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0071_1_fix_invalid_filename_for_0072.xml
      (date 1767469155355)
   > @@ -30,17 +30,14 @@
   >                 author="fineract"
   >                 runOnChange="false"
   >                 runAlways="false">
   > -
   >          <comment>
   >              Fix invalid space in Liquibase changelog filename for Windows 
compatibility
   >          </comment>
   > -
   > -        <sql>
   > -            UPDATE databasechangelog
   > -            SET filename = 
'db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml'
   > -            WHERE filename = 
'db/changelog/tenant/parts/0072_add_result_and status_to_command_source.xml';
   > -        </sql>
   > -
   > +        <update tableName="DATABASECHANGELOG">
   > +            <column name="filename" 
value="db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml"/>
   > +            <column name="md5sum" />
   > +            <where>filename = 
'db/changelog/tenant/parts/0072_add_result_and 
status_to_command_source.xml'</where>
   > +        </update>
   >      </changeSet>
   >  
   >  </databaseChangeLog>
   > Index: 
fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml
   > IDEA additional info:
   > Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
   > <+>UTF-8
   > ===================================================================
   > diff --git 
a/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml 
b/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml
   > --- 
a/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml 
       (revision 6187378e83873aaf3fe911f00f9ea085460dd795)
   > +++ 
b/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml 
       (date 1767469466837)
   > @@ -91,6 +91,7 @@
   >      <include 
file="parts/0069_add_unique_constraint_for_reversal_external_id_of_loan_transactions.xml"
 relativeToChangelogFile="true"/>
   >      <include 
file="parts/0070_add_event_configuration_for_delinquency_range_change_event.xml"
 relativeToChangelogFile="true"/>
   >      <include 
file="parts/0071_add_external_id_support_for_loan_transaction.xml" 
relativeToChangelogFile="true"/>
   > +    <include file="parts/0071_1_fix_invalid_filename_for_0072.xml" 
relativeToChangelogFile="true"/>
   >      <include 
file="parts/0072_add_result_and_status_to_command_source.xml" 
relativeToChangelogFile="true" />
   >      <include 
file="parts/0073_add_result_status_code_to_command_source.xml" 
relativeToChangelogFile="true" />
   >      <include 
file="parts/0074_add_last_cob_business_date_column_to_loan.xml" 
relativeToChangelogFile="true"/>
   > Index: 
fineract-provider/src/main/resources/db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml
   > IDEA additional info:
   > Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
   > <+>UTF-8
   > ===================================================================
   > diff --git 
a/fineract-provider/src/main/resources/db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml
 
b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml
   > --- 
a/fineract-provider/src/main/resources/db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml
      (revision 6187378e83873aaf3fe911f00f9ea085460dd795)
   > +++ 
b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml
      (date 1767469344634)
   > @@ -24,12 +24,14 @@
   >                     
xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog 
http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-4.1.xsd";>
   >  
   >      <changeSet id="1" author="fineract" context="postgresql">
   > +        <validCheckSum>9:b1a2065c6c2f9fba5035e700435a80ab</validCheckSum>
   >          <renameColumn newColumnName="status"
   >                         oldColumnName="processing_result_enum"
   >                         tableName="m_portfolio_command_source"/>
   >      </changeSet>
   >  
   >      <changeSet id="2" author="fineract" context="mysql">
   > +        <validCheckSum>9:9c4456b507915a0ee7a16edca4cddb46</validCheckSum>
   >          <renameColumn newColumnName="status"
   >                        oldColumnName="processing_result_enum"
   >                        tableName="m_portfolio_command_source"
   > @@ -37,6 +39,7 @@
   >      </changeSet>
   >  
   >      <changeSet id="3" author="fineract">
   > +        <validCheckSum>9:b5a7d85155d23aadde3403c2d0a8c75d</validCheckSum>
   >          <addColumn tableName="m_portfolio_command_source">
   >              <column name="result" type="TEXT"/>
   >          </addColumn>
   > ```
   
   @adamsaghy I checked changelog-tenant.xml and the line <include 
file="parts/0071_1_fix_invalid_filename_for_0072.xml" 
relativeToChangelogFile="true"/> is already present on line 94. The other two 
fixes (changing <sql> to <update> and adding validCheckSum tags) have been 
applied. Thank you for your guidance.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to