Skip to content
This repository was archived by the owner on Nov 24, 2021. It is now read-only.

Fix create more then 2 universes#304

Open
jkroepke wants to merge 2 commits into
masterfrom
issue-301
Open

Fix create more then 2 universes#304
jkroepke wants to merge 2 commits into
masterfrom
issue-301

Conversation

@jkroepke

@jkroepke jkroepke commented Jun 5, 2017

Copy link
Copy Markdown
Owner

Fixes #301

@ghost ghost left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This Fix Don't Work for Version 1.8 2moons. The same issue comes up.
2018-12-27

@ghost

ghost commented Dec 27, 2018

Copy link
Copy Markdown

Here is the file with the suggested edit to the ShowUniversePage.php
ShowUniversePage.zip

@wrath76

wrath76 commented Dec 27, 2018 via email

Copy link
Copy Markdown

@ghost

ghost commented Dec 27, 2018

Copy link
Copy Markdown

I quick fix is to copy your config in the database call 2 for something and them go into it and edit it....rhe system will see 2 universes

On Thu, Dec 27, 2018, 14:08 MadDog402 @.*** wrote: Here is the file with the suggested edit to the ShowUniversePage.php ShowUniversePage.zip https://github.com/jkroepke/2Moons/files/2713132/ShowUniversePage.zip — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#304 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AVE5aOE4jYf86Tg8JMOjTfcZYYMt30_vks5u9Rq0gaJpZM4Nwptd .

Well i have two universes. it's when i go to make more then 2 i have the issue. It has something to do with the coding for that message to pop up every time i go to create a 3rd universe.

@wrath76

wrath76 commented Dec 27, 2018 via email

Copy link
Copy Markdown

@ghost

ghost commented Dec 27, 2018

Copy link
Copy Markdown

Thank's that worked. Once i'm more fluent in PHP i will be fixing that so i don't have to go into the database.

@jkroepke

Copy link
Copy Markdown
Owner Author

I add an additional fix. Can someone test it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants