Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean up mentions of OTOBO 5 and OTOBO 6 #232

Closed
bschmalhofer opened this issue Jul 22, 2020 · 7 comments
Closed

Clean up mentions of OTOBO 5 and OTOBO 6 #232

bschmalhofer opened this issue Jul 22, 2020 · 7 comments
Assignees
Labels
question Further information is requested
Milestone

Comments

@bschmalhofer
Copy link
Contributor

bschmalhofer commented Jul 22, 2020

In these cases it can be presumed that either legacy version of OTRS or OTOBO 10 is meant.

@bschmalhofer
Copy link
Contributor Author

bschmalhofer commented Jul 22, 2020

In the migration modules and tests there are frequent mentions of OTOBO 5 and OTOBO 6. This is confusing as OTOBO starts with Version 10.
I checked with the following commands:

  • grep -r -i 'otobo\s*[0123456789]' . | grep -v -i 'otobo\s*10'
  • find . -iname '*OTOBO[0123456789]*'

I could do the renaming, but I would need some clarification concerning the migration pathes. OTRS 5 => OTOBO 10 vs, OTRS 6 => OTOBO 10.

@bschmalhofer
Copy link
Contributor Author

Note that the only relevant migration is from OTRS 6 to OTOBO 10.

bschmalhofer added a commit that referenced this issue Aug 17, 2020
In OTRS 6 the xml files are already in the folder XML.
Therefore don't put the migrated files into the subdir 'XML'.
Instead create a backup file.
bschmalhofer added a commit that referenced this issue Aug 17, 2020
Just because the log message is not really needed and DESTROY()
is a potential source of errors.
bschmalhofer added a commit that referenced this issue Aug 17, 2020
…MLStructure()

Only the renaming of the root tag remains.
Everything else concerned only OTRS 5 to OTRS 6.
Added a test case for Dev::Tools::Migrate::ConfigXMLStructur
bschmalhofer added a commit that referenced this issue Aug 17, 2020
This module was only concerned about OTRS 5 -> OTRS 6 migration
bschmalhofer added a commit that referenced this issue Aug 17, 2020
@bschmalhofer
Copy link
Contributor Author

The UnitTests look fine. Closing.

@bschmalhofer
Copy link
Contributor Author

Should this be backported to the rel-10_0 branch? Already for 10.0.4 ?

@StefanRother-OTOBO
Copy link
Contributor

Hi,

I think this UnitTests are for migration OTRS 5 to 6, cause I do not create UnitTests for migrating to OTOBO yet.
For me it's fine to remove the tests.

Thanks,

Stefan

bschmalhofer added a commit that referenced this issue Oct 12, 2020
In OTRS 6 the xml files are already in the folder XML.
Therefore don't put the migrated files into the subdir 'XML'.
Instead create a backup file.
bschmalhofer added a commit that referenced this issue Oct 12, 2020
Just because the log message is not really needed and DESTROY()
is a potential source of errors.
bschmalhofer added a commit that referenced this issue Oct 12, 2020
…MLStructure()

Only the renaming of the root tag remains.
Everything else concerned only OTRS 5 to OTRS 6.
Added a test case for Dev::Tools::Migrate::ConfigXMLStructur
bschmalhofer added a commit that referenced this issue Oct 12, 2020
This module was only concerned about OTRS 5 -> OTRS 6 migration
bschmalhofer added a commit that referenced this issue Oct 12, 2020
@bschmalhofer
Copy link
Contributor Author

I cherry-picked the commits for this branch into the rel-10_0 branch. Let's close this issue again when things look fine again. There might be some fallout from resolving the merge conflicts.

bschmalhofer added a commit that referenced this issue Oct 14, 2020
Issue #232: fix contributor otobo365.com -> otrs365.com
@bschmalhofer
Copy link
Contributor Author

Did a final check on rel-10_0 and repaired info@otrs365.com. Closing this issue again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants