? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
14 Jan 2017

Pull Request for Issue #13551 .

  • rebuild #__assets in installation sql for (mysql,postgresql,mssql)
  • rebuild #__assets in sample_testing sql for (mysql,postgresql,mssql)
  • rebuild #__assets in sample_blog sql for (mysql,postgresql,mssql)
  • rebuild #__assets in sample_data sql for (mysql,postgresql,mssql)
  • rebuild #__assets insample_learn sql for (mysql,postgresql,mssql)
  • rebuild #__assets in sample_brochure sql for (mysql,mssql)

Summary of Changes

remove old components from the #__assets table like com_massmail
rebuild #__assets for sql installation file

Test instructions

install a fresh joomla site with sample datasets all work as before

avatar alikon alikon - open - 14 Jan 2017
avatar alikon alikon - change - 14 Jan 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Jan 2017
Category Administration com_admin
avatar sanderpotjer
sanderpotjer - comment - 14 Jan 2017

I have tested this item ? unsuccessfully on 7165668

Script is currently not working, missing a semicolon on line 1825 of administrator/components/com_admin/script.php


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

avatar sanderpotjer sanderpotjer - test_item - 14 Jan 2017 - Tested unsuccessfully
avatar sanderpotjer sanderpotjer - test_item - 14 Jan 2017 - Tested unsuccessfully
avatar alikon alikon - change - 14 Jan 2017
Labels Added: ?
avatar alikon
alikon - comment - 14 Jan 2017

damned c&p ;)

avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Jan 2017

Afaik You need to remove it instalattion sql too

avatar joomla-cms-bot joomla-cms-bot - change - 15 Jan 2017
Category Administration com_admin Administration com_admin Postgresql SQL Installation
avatar sanderpotjer
sanderpotjer - comment - 15 Jan 2017

There are more references to the com_massmail in the installation sql files, like the sample_blog, sample_brochure, sample_data, sample_learn and sample_testing. And of course for the sqlazure, mysql and postgresql versions. This should be removed too.

avatar alikon
alikon - comment - 15 Jan 2017

i know, i need to find a bit free time
consider a work in progress for now

avatar joomla-cms-bot joomla-cms-bot - change - 17 Jan 2017
Category Administration com_admin Postgresql SQL Installation Administration com_admin SQL Installation Postgresql MS SQL
avatar alikon alikon - change - 19 Jan 2017
The description was changed
avatar alikon alikon - edited - 19 Jan 2017
avatar alikon alikon - change - 19 Jan 2017
The description was changed
avatar alikon alikon - edited - 19 Jan 2017
avatar alikon alikon - change - 21 Jan 2017
The description was changed
avatar alikon alikon - edited - 21 Jan 2017
avatar alikon alikon - change - 21 Jan 2017
The description was changed
avatar alikon alikon - edited - 21 Jan 2017
avatar alikon alikon - change - 21 Jan 2017
The description was changed
avatar alikon alikon - edited - 21 Jan 2017
avatar alikon alikon - change - 21 Jan 2017
Title
remove old components from assets like com_massmail
[installation] - rebuild assets table removing com_massmail
avatar alikon alikon - change - 21 Jan 2017
Title
remove old components from assets like com_massmail
[installation] - rebuild assets table removing com_massmail
avatar alikon alikon - edited - 21 Jan 2017
avatar alikon
alikon - comment - 21 Jan 2017

it's ready for test now

avatar marrouchi
marrouchi - comment - 24 Jan 2017

@alikon Can please provice clear test instructions ? I wasn't able to apply the patch on a fresh clone of the repo using git apply --check 13587.patch

avatar alikon
alikon - comment - 24 Jan 2017
  1. install latest staging with one of the sample data provided or with no data (multilingual)
    2.a install com_patchtester and apply this patch or 2.b (i'm not a git expert)
    2.b git apply --check 13587.patch
  2. delete configuration.php
  3. install again with the same data choosed at 1.)
  4. check that all work as before
  5. check that in the #__assets table there are no occurrencies with com_massmail

repeat 1 thru 6 for all kind of sample data

avatar marrouchi
marrouchi - comment - 25 Jan 2017

I have tested this item successfully on 217edaa

Thank you for the test instructions !

Tested on mysql DB.

Note that database is clean but there still occurrencies in files.


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

avatar marrouchi marrouchi - test_item - 25 Jan 2017 - Tested successfully
avatar alikon
alikon - comment - 28 Jan 2017

can you specify

there still occurrencies in files.

avatar marrouchi
marrouchi - comment - 28 Jan 2017

Sorry, ignore my last comment. They are occurencies in com_users that has a mass mail feature.

avatar csthomas
csthomas - comment - 2 Feb 2017

I have tested this item successfully on d8c76e5

Mysql and postgresql


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

avatar csthomas csthomas - test_item - 2 Feb 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Feb 2017

at Point 3 got:
bildschirmfoto 2017-02-22 um 19 14 10 kopie

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 May 2017

I have tested this item ? unsuccessfully on d8c76e5

  • /index.php/single-news-feed – 404 Error. Feed not found
  • /index.php/feed-display – Feed not found.
    bildschirmfoto 2017-05-28 um 10 41 51

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13587.
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 28 May 2017 - Tested unsuccessfully
avatar alikon alikon - change - 3 Oct 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-10-03 06:22:28
Closed_By alikon
avatar alikon alikon - close - 3 Oct 2017

Add a Comment

Login with GitHub to post a comment