Re: [Tiki-devel] Tiki | Failed pipeline for master | 7fe28a3f

Hi all again (Ricardo really i guess ;)

Have added a hopeful fix in 

    https://gitlab.com/tikiwiki/tiki/-/commit/8bb38c69

Pipeline running, fingers crossed... well, still fails but with fewer errors, so i must be on the right track! 🤓

I don't completely understand the code that is supposed "to account for the case where an old entry is removed" in tiki_menu_options and tiki_actionlog_conf tables, presumably this is to renumber the auto-increment columns so they match? Do i need to add the same thing for tiki_live_support_modules do you think?

But why is there still one error on tiki_menu_options i wonder?

Out of time for this now, will try again more tomorrow...

jonny



> On 20 Oct 2021, at 16:37, Jonny Bradley via TikiWiki-devel <[email protected]> wrote:
> 
> Hi all
> 
> Oops, sorry, i borked the tests! :o
> 
> I removed the references from tiki.sql for new installs but deliberately didn't write an updater script to do the same on upgrades in case the image gallery migration went wrong, for instance.
> 
> How should we handle this? Do i need to add some exceptions to the compare script, presumably in doc/devtools/check_schema_upgrade.php somewhere (which i've never managed to get to work locally in the past)...
> 
> Ah, presumably i should add all this in \TikiDevTools\CheckSchemaUpgrade::scrubDbCleanThingsThatShouldChange?
> 
> Or should i add an upgrade script to actually remove that data?
> 
> TIA
> 
> jonny
> 
> 
> 
> 
> 
>> Begin forwarded message:
>> 
>> From: GitLab <[email protected]>
>> Subject: Tiki | Failed pipeline for master | 7fe28a3f
>> Date: 20 October 2021 at 16:25:44 BST
>> Reply-To: GitLab <[email protected]>
>> 
>> 
>> 	Pipeline #392074467 has failed!
>> 
>> Project	Tiki Wiki CMS Groupware / Tiki
>> Branch	
>> 	master
>> Commit	
>> 	7fe28a3f
>> [KIL] imagegallery: Even more left overs remove...
>> Commit Author	
>> 	Jonny Bradley
>> 
>> Pipeline #392074467 triggered by		Jonny Bradley
>> had 14 failed jobs.
>> Failed jobs
>> 	tiki-check
>> check-directories
>> 	lint
>> smarty-lint
>> 	tiki-check
>> db-upgrade-18-lts-myisam
>> 	lint
>> phpcs
>> 	tiki-check
>> db-upgrade-19
>> 	tiki-check
>> db-upgrade-18-lts
>> 	tiki-check
>> db-upgrade-20
>> 	tiki-check
>> db-upgrade-21
>> 	tiki-check
>> sql-engine-conversion
>> 	tiki-check
>> db-upgrade-22
>> 	tiki-check
>> translation-standards
>> 	tiki-check
>> db-upgrade-22-mysql8
>> 	tiki-check
>> db-upgrade-21-mysql8
>> 	tiki-check
>> db-upgrade-20-mysql8
>> 
>> You're receiving this email because of your account on gitlab.com. Manage all notifications · Help
> 
> 
> 
> _______________________________________________
> TikiWiki-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel



_______________________________________________
TikiWiki-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel