search

sulu / sulu-document-manager

Sulu document manager
sulu.io
6 stars 7 forks source link

Fixed auto-name subscriber to rename at the very end of persist #83

Closed wachterjohannes closed 8 years ago

wachterjohannes commented 8 years ago

This PR fixes the AutoNameSubscriber to rename the node at the very en of persist.

Newly created issue with phpbench https://github.com/sulu/sulu-document-manager/issues/84

wachterjohannes commented 8 years ago

@danrot fixed

danrot commented 8 years ago

I would also add a function test for that.

danrot commented 8 years ago

Is that now ready to review? Because you've said yesterday that you were still experiencing a bug.

wachterjohannes commented 8 years ago

when you sulu tests are green this is also good!