Conflicting Files ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
9 Jan 2021

Pull Request for Issue #31898 .

Summary of Changes

added missing new cors settings default on install

Testing Instructions

  1. install joomla
  2. run php cli/joomla.php config:set error_reporting=development

Actual result BEFORE applying this Pull Request

error

Expected result AFTER applying this Pull Request

No error should be present

Documentation Changes Required

avatar alikon alikon - open - 9 Jan 2021
avatar alikon alikon - change - 9 Jan 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Jan 2021
Category Installation
avatar richard67
richard67 - comment - 9 Jan 2021

Forgotten to change the settings after copying?

avatar richard67
richard67 - comment - 9 Jan 2021

Seems to be copied cache settings now.

0a51ec2 9 Jan 2021 avatar alikon oops
avatar alikon alikon - change - 9 Jan 2021
Labels Added: ?
avatar alikon
alikon - comment - 9 Jan 2021

my bad
fixed

avatar toivo
toivo - comment - 9 Jan 2021

I have tested this item successfully on 0a51ec2

Tested successfully in Beta7-dev of 9 January.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31937.

avatar toivo toivo - test_item - 9 Jan 2021 - Tested successfully
avatar Razzo1987
Razzo1987 - comment - 9 Jan 2021

Keep returning the error:

Symfony\Component\ErrorHandler\Error\UndefinedFunctionError^ {#142
  #message: "Attempted to call function "simplexml_load_file" from namespace "Joomla\CMS\Language"."
  #code: 0
  #file: "./libraries/src/Language/LanguageHelper.php"
  #line: 680
  trace: {
    ./libraries/src/Language/LanguageHelper.php:680 {
      Joomla\CMS\Language\LanguageHelper::parseXMLLanguageFile($path)^
      › // Try to load the file
      › $xml = simplexml_load_file($path);
      › 
    }
    ./libraries/src/Language/LanguageHelper.php:572 { …}
    ./libraries/src/Language/Language.php:193 { …}
    ./libraries/src/Language/LanguageFactory.php:32 { …}
    ./libraries/src/Language/CachingLanguageFactory.php:43 { …}
    ./libraries/src/Service/Provider/Application.php:121 { …}
    ./libraries/vendor/joomla/di/src/ContainerResource.php:176 { …}
    ./libraries/vendor/joomla/di/src/Container.php:96 { …}
    ./cli/joomla.php:67 { …}
  }
}

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31937.
avatar wilsonge
wilsonge - comment - 10 Jan 2021

@Razzo1987 looks like your PHP setup is missing lib-xml

avatar gostn
gostn - comment - 10 Jan 2021

I have tested this item successfully on 0a51ec2


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31937.

avatar gostn gostn - test_item - 10 Jan 2021 - Tested successfully
avatar alikon alikon - change - 10 Jan 2021
Status Pending Ready to Commit
avatar alikon
alikon - comment - 10 Jan 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31937.

avatar wilsonge
wilsonge - comment - 11 Jan 2021

This isn't going to be enough. We're going to need to cover configuration.php files for people upgrading from Joomla 3.10

avatar alikon alikon - change - 11 Jan 2021
Status Ready to Commit Pending
avatar alikon
alikon - comment - 11 Jan 2021

back to pending for cover update from 3.10


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31937.

avatar brianteeman
brianteeman - comment - 11 Jan 2021

This isn't going to be enough. We're going to need to cover configuration.php files for people upgrading from Joomla 3.10

Don't they automatically get the cors settings added to the configuration.php as soon as they save the global config for the first time AND as its disabled by default it doesnt matter if they dont do it

avatar wilsonge
wilsonge - comment - 12 Jan 2021

Don't they automatically get the cors settings added to the configuration.php as soon as they save the global config for the first time AND as its disabled by default it doesnt matter if they dont do it

For the actual API yes, but per the original issue apparently something breaks inside com_config CLI endpoint

avatar joomla-cms-bot joomla-cms-bot - change - 17 Jan 2021
Category Installation Administration com_admin Installation
avatar wilsonge
wilsonge - comment - 18 Jan 2021

That looks better but it needs to cover cases on further updates where these values are already set

avatar alikon
alikon - comment - 18 Jan 2021
public function postflight($action, $installer)
{
	if ($action !== 'update')
	{
		return true;
	}


	if (empty($this->fromVersion) || version_compare($this->fromVersion, '4.0.0', 'ge'))
	{
		return true;
	}

the postflight runs on update from version less than 4.0.0

avatar richard67
richard67 - comment - 18 Jan 2021

I.e. it never runs when updating a previous beta which doesn't have the new settings?

avatar alikon
alikon - comment - 18 Jan 2021

it runs on previous beta too....
as an example

$old='4.0.0-beta4-dev';
version_compare($old, '4.0.0', 'ge');  // return false
avatar alikon alikon - change - 26 Jun 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-06-26 08:09:27
Closed_By alikon
Labels Added: Conflicting Files
avatar alikon alikon - close - 26 Jun 2021

Add a Comment

Login with GitHub to post a comment