? Pending

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
12 Aug 2016

Summary of Changes

Joomla on install articles, categories etc with a created date with more than 5 years (2011-01-01) which does not look good when you are installing something new.

So this PR adds a method to the installation process to update core and sample data created dates to the current date.

Also corrects category creator id in com_banners and com_user_notes (was missing).

Testing Instructions

  1. Use latest staging
  2. Delete configuration.php, refresh the page and install joomla with any sample data install step
  3. Check the created dates of articles, categories, contacts, etc
  4. Now apply this patch, delete configuration.php, refresh the page and install joomla
  5. Repeat step 2 to 3 check the created date is now the Joomla install date.
  6. Do a code review.
avatar joomla-cms-bot joomla-cms-bot - change - 12 Aug 2016
Category Installation
avatar andrepereiradasilva andrepereiradasilva - open - 12 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 12 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Aug 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 12 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 12 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 12 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 12 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 12 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 12 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - edited - 12 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 12 Aug 2016
Title
[installation] Update created date plus minor corrections
[installation] Created in 2011-01-01? Update created date plus minor corrections
avatar andrepereiradasilva andrepereiradasilva - change - 12 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 12 Aug 2016
avatar brianteeman
brianteeman - comment - 13 Aug 2016

I understand what you are doing with this and its a good idea
The only small issue is with the testing data (which is no longer shipped with the cms and is only in the github download) where we do need to keep the existing dates or all the articles get the same date and it will be hard to test ordering etc.

So is it possible to exclude running updateCreatedDates when you have selected that dataset

avatar andrepereiradasilva
andrepereiradasilva - comment - 13 Aug 2016

ok i understood
will check that

avatar brianteeman
brianteeman - comment - 13 Aug 2016

Thanks - and sorry if I am making work for you but its needed

avatar andrepereiradasilva
andrepereiradasilva - comment - 13 Aug 2016

np. will chek when i have time

avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Aug 2016

ok @brianteeman sample_testing is now excluded. please test.

avatar alikon
alikon - comment - 14 Aug 2016

i've not tested exactly as described

  • fresh install with current staging + blog data
  • enabled & installed multilanguage (4) content too

it seems ok
currentdate

but noticed some incongruent date (Modified Date < Created Date)

inconsistentdates

and for article is more incongruent (Start Publishing < Modified Date < Created Date )

inco2

avatar brianteeman
brianteeman - comment - 14 Aug 2016

Good spot @alikon . the code is only updating the created_by date

avatar joomla-cms-bot joomla-cms-bot - change - 14 Aug 2016
Category Installation
avatar joomla-cms-bot joomla-cms-bot - change - 14 Aug 2016
Category Installation
af78020 14 Aug 2016 avatar andrepereiradasilva cs
avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Aug 2016

ok, now should update all the dates (except null dates)
please test install with/without sampledata and in mysql and postgresql.

avatar brianteeman
brianteeman - comment - 14 Aug 2016

Dates looked ok. Installed with blog data
JUser: :_load: Unable to load user with ID: 713

in com_content the created_by is correct (in my case132) but the modified_by is wrong with 713


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Aug 2016

JUser: :_load: Unable to load user with ID: 713

that as nothing to do with this PR

in com_content the created_by is correct (in my case132) but the modified_by is wrong with 713

that as nothing to do with this PR

avatar joomla-cms-bot joomla-cms-bot - change - 5 Sep 2016
Category Installation
avatar joomla-cms-bot joomla-cms-bot - change - 5 Sep 2016
Category Installation
avatar wilsonge
wilsonge - comment - 5 Sep 2016

Looks good to me now. Can I have one person quickly retest this just to make sure there are no new issues with the rebase? Thanks!

avatar brianteeman
brianteeman - comment - 5 Nov 2016

Making a note for myself to test again tonight

avatar brianteeman brianteeman - test_item - 5 Nov 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 5 Nov 2016

I have tested this item successfully on b36c756


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

avatar wilsonge wilsonge - close - 8 Nov 2016
avatar wilsonge wilsonge - close - 8 Nov 2016
avatar wilsonge wilsonge - merge - 8 Nov 2016
avatar wilsonge wilsonge - reference | 123821a - 8 Nov 16
avatar wilsonge wilsonge - merge - 8 Nov 2016
avatar wilsonge wilsonge - close - 8 Nov 2016
avatar wilsonge wilsonge - change - 8 Nov 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-11-08 10:58:28
Closed_By wilsonge
avatar wilsonge wilsonge - change - 8 Nov 2016
Status Fixed in Code Base Pending
avatar wilsonge wilsonge - change - 8 Nov 2016
Milestone Added:
avatar wilsonge
wilsonge - comment - 8 Nov 2016

Thanks Andre!

Add a Comment

Login with GitHub to post a comment