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

Can't save Masquerade fields after edit #6

Closed
benduckwdesign opened this issue Jun 19, 2017 · 20 comments
Closed

Can't save Masquerade fields after edit #6

benduckwdesign opened this issue Jun 19, 2017 · 20 comments
Assignees

Comments

@benduckwdesign
Copy link

When I try to click on the "Edit field" button after making some changes to a field, an error pops up at the bottom saying 'Oops! Something went wrong. Please reload the page and try again.'

I would provide an error log but I'm not sure where to find it for this specific extension.
I'm using version 0.1.0-beta.6 of Flarum, PHP 5.6.25, and 0.1.0 of Masquerade.

Any help is appreciated.

@luceos
Copy link
Contributor

luceos commented Jun 20, 2017

Can you enable debug and see what the debug buttons on the woops modal gives as information? Otherwise the logs can be found in storage/logs/flarum.log.

@benduckwdesign
Copy link
Author

benduckwdesign commented Jun 20, 2017

Yeah, I didn't see anything recent in the flarum.log so I'll enable debug...

Okay, upon further inspection it seems this is the problem:

"errors": [ { "code": 500, "title": "Internal server error", "detail": "exception 'PDOException' with message 'SQLSTATE[42S22]: Column not found: 1054 Unknown column 'excerpt_length' in 'field list'' in [omitted]/vendor/illuminate/database/Connection.php:411

The rest of the error is just a stack trace (there aren't any other errors.)
I'm not entirely sure why I would have this issue when every other extension works fine, as well as posting as usual, but no one else has reported this problem either.

@benduckwdesign
Copy link
Author

Okay, upon even further inspection, the delete field button does not work either. It gives no error, though. Strangely, the trash bin button at the top does work, which is the button I have been using to delete the fields.

@luceos
Copy link
Contributor

luceos commented Jun 20, 2017

excerpt length is not part of this extension.. Can you run php flarum migrate or disable all other extensions temporarily?

@benduckwdesign
Copy link
Author

Running php flarum migrate gives me an error PHP Fatal error: Uncaught PDOException: SQLSTATE[HY000] [2002] No such file or directory in [omitted]/vendor/illuminate/database/Connectors/Connector.php:55. I'll try disabling all my other extensions.

@benduckwdesign
Copy link
Author

benduckwdesign commented Jun 20, 2017

Okay, I disabled all other extensions except for my other Flagrow extensions and the ones that come with Flarum by default. No error is shown after clicking Edit Field now, but the delete field button does still not work, yet the trash bin does. I'm going to re-enable the extensions one by one in debug mode to see which one is conflicting.

It appears the edit button is working but there's no notification or anything.

@benduckwdesign
Copy link
Author

Okay, I figured out which extension is causing the problem.

It appears to be https://github.com/jordanjay29/flarum-ext-summaries which is causing the issue.

@benduckwdesign
Copy link
Author

benduckwdesign commented Jun 20, 2017

Alright, I enabled all other extensions except ext-summaries and your plugin works perfectly.
Since summaries doesn't really conflict with masquerade anywhere else, I suppose a temporary workaround would be to disable summaries whenever I need to edit the profile page template.

I would drop an issue on the summaries repo but I would think that it is best left to the extension maintainers to figure out where the extensions conflict, yeah? I wouldn't really be of any help at this point.

Thank you for your time and your wonderful plugin!

EDIT: Turns out I also had a link error to the mySQL socket which I have fixed now.

@luceos
Copy link
Contributor

luceos commented Jun 21, 2017

Now that you fixed that socket, try running migrate again.

/ping @jordanjay29 for his summaries extension, perhaps he can give you additional suggestions on this problem.

@benduckwdesign
Copy link
Author

I ran php flarum migrate which worked successfully this time due to the fixed socket, but the same missing column error persists when both summaries and masquerade are enabled.

@jordanjay29 Any ideas?

@luceos
Copy link
Contributor

luceos commented Jun 22, 2017

Hi @miniluigi008 I've already had a chat with Jordan, we both found nothing that might point to the cause of this issue. Can you post the full error "trace" from storage/log/flarum.log?

@benduckwdesign
Copy link
Author

Here is the flarum log, there seem to be only two traces related to this issue in it, even though I experienced it more than twice.
flarum log.txt

Here is the full error trace from the "Oops!" Debug selection on the web.
Debug Error.txt

@luceos
Copy link
Contributor

luceos commented Jun 22, 2017

For your information the interesting part is this:

#9 /[omitted webroot]/forums/vendor/flagrow/masquerade/src/Repositories/FieldRepository.php(66): Illuminate\\Database\\Eloquent\\Model->save()\n
#10 /[omitted webroot]/forums/vendor/flagrow/masquerade/src/Api/Controllers/SaveFieldController.php(55): Flagrow\\Masquerade\\Repositories\\FieldRepository->findOrNew(Array)

I'll look into it. I might have an idea where this comes from.

@luceos luceos added bug and removed question labels Jun 22, 2017
@luceos luceos self-assigned this Jun 22, 2017
@luceos
Copy link
Contributor

luceos commented Jun 22, 2017

@miniluigi008 one last thing, can you post your require section of the composer.json ;)

@benduckwdesign
Copy link
Author

    "require": {
        "flarum/core": "^0.1.0",
        *"flarum/flarum-ext-akismet": "^0.1.0",
        "flarum/flarum-ext-approval": "^0.1.0",
        *"flarum/flarum-ext-auth-facebook": "^0.1.0",
        *"flarum/flarum-ext-auth-github": "^0.1.0",
        *"flarum/flarum-ext-auth-twitter": "^0.1.0",
        "flarum/flarum-ext-bbcode": "^0.1.0",
        "flarum/flarum-ext-emoji": "^0.1.0",
        "flarum/flarum-ext-english": "^0.1.0",
        "flarum/flarum-ext-flags": "^0.1.0",
        *"flarum/flarum-ext-likes": "^0.1.0",
        "flarum/flarum-ext-lock": "^0.1.0",
        "flarum/flarum-ext-markdown": "^0.1.0",
        "flarum/flarum-ext-mentions": "^0.1.0",
        *"flarum/flarum-ext-pusher": "^0.1.0",
        "flarum/flarum-ext-sticky": "^0.1.0",
        "flarum/flarum-ext-subscriptions": "^0.1.0",
        "flarum/flarum-ext-suspend": "^0.1.0",
        "flarum/flarum-ext-tags": "^0.1.0",
        "zendframework/zend-stratigility": "1.2.*",
        "datitisev/flarum-ext-dashboard": "^0.1.0@beta",
        "flagrow/flarum-ext-latex": "^0.1.1",
        "sijad/flarum-ext-links": "^0.1.0@beta",
        "avatar4eg/flarum-ext-users-list": "^0.1.1",
        "s9e/flarum-ext-mediaembed": "^0.3.2",
        *"davis/flarum-ext-securehttps": "^0.1.0@beta",
        "davis/flarum-ext-inviteonly": "^0.1.0@beta",
        "flagrow/byobu": "^0.1.0@beta",
        *"antoinefr/flarum-ext-money": "^0.3.0",
        "flagrow/upload": "0.5.7",
        "sijad/flarum-ext-details": "^0.1.2",
        "jordanjay29/flarum-ext-summaries": "^0.2.0",
        "flagrow/masquerade": "^0.1.0",
        "wiwatsrt/flarum-ext-best-answer": "^0.1.0@beta",
        "davis/flarum-ext-split": "^0.1.0@beta",
        "clarkwinkelmann/flarum-ext-circle-groups": "^0.1.0@beta",
        "oe800/flarum-ext-bbcode-special": "^0.1.4",
        "reflar/gamification": "^1.0@RC",
        *"oe800/flarum-gravatar": "^0.1.2",
        "oe800/flarum-ext-bbcode-cards": "^0.1.4",
        "oe800/flarum-ext-bbcode-alerts": "^0.1.5",
        "oe800/flarum-ext-bbcode-bars": "^0.1.5",
        "johnhearfield/flarum-ext-landing-page": "^0.1.0@beta",
        "clarkwinkelmann/flarum-ext-emojionearea": "^0.1.1"
    }

I've put an asterisk by the ones that are currently disabled. As you can see, I'm a big fan of flagrow extensions. 😏

@luceos
Copy link
Contributor

luceos commented Jun 22, 2017

Great thanks! Cool to see that abandoned latex extension still works 😊

@luceos
Copy link
Contributor

luceos commented Jun 22, 2017

So the only extension of all of these is summaries that uses excerpt_length:

image

@luceos
Copy link
Contributor

luceos commented Jul 10, 2017

I still think this is unrelated to this extension and so will close this issue here.

@luceos luceos closed this as completed Jul 10, 2017
@luceos luceos removed the bug label Jul 10, 2017
@luceos
Copy link
Contributor

luceos commented Jul 19, 2017

Okay found the issue!

@benduckwdesign
Copy link
Author

Thank you! @jordanjay29 @luceos

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

No branches or pull requests

2 participants