{"action":"created","issue":{"url":"https://api.github.com/repos/Sylius/Sylius/issues/3914","labels_url":"https://api.github.com/repos/Sylius/Sylius/issues/3914/labels{/name}","comments_url":"https://api.github.com/repos/Sylius/Sylius/issues/3914/comments","events_url":"https://api.github.com/repos/Sylius/Sylius/issues/3914/events","html_url":"https://github.com/Sylius/Sylius/issues/3914","id":127383468,"number":3914,"title":"AttributeValue __toString() invalid","user":{"login":"peteward","id":7068515,"avatar_url":"https://avatars.githubusercontent.com/u/7068515?v=3","gravatar_id":"","url":"https://api.github.com/users/peteward","html_url":"https://github.com/peteward","followers_url":"https://api.github.com/users/peteward/followers","following_url":"https://api.github.com/users/peteward/following{/other_user}","gists_url":"https://api.github.com/users/peteward/gists{/gist_id}","starred_url":"https://api.github.com/users/peteward/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peteward/subscriptions","organizations_url":"https://api.github.com/users/peteward/orgs","repos_url":"https://api.github.com/users/peteward/repos","events_url":"https://api.github.com/users/peteward/events{/privacy}","received_events_url":"https://api.github.com/users/peteward/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-19T07:33:32Z","updated_at":"2016-01-19T07:52:27Z","closed_at":null,"body":"Since the value is typed this no longer works. You could cast with (string) but this probably won't have the desired affect on date fields so it should probably just be removed?\r\n\r\nhttps://github.com/Sylius/Sylius/blob/master/src/Sylius/Component/Attribute/Model/AttributeValue.php#L75"},"comment":{"url":"https://api.github.com/repos/Sylius/Sylius/issues/comments/172768233","html_url":"https://github.com/Sylius/Sylius/issues/3914#issuecomment-172768233","issue_url":"https://api.github.com/repos/Sylius/Sylius/issues/3914","id":172768233,"user":{"login":"peteward","id":7068515,"avatar_url":"https://avatars.githubusercontent.com/u/7068515?v=3","gravatar_id":"","url":"https://api.github.com/users/peteward","html_url":"https://github.com/peteward","followers_url":"https://api.github.com/users/peteward/followers","following_url":"https://api.github.com/users/peteward/following{/other_user}","gists_url":"https://api.github.com/users/peteward/gists{/gist_id}","starred_url":"https://api.github.com/users/peteward/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peteward/subscriptions","organizations_url":"https://api.github.com/users/peteward/orgs","repos_url":"https://api.github.com/users/peteward/repos","events_url":"https://api.github.com/users/peteward/events{/privacy}","received_events_url":"https://api.github.com/users/peteward/received_events","type":"User","site_admin":false},"created_at":"2016-01-19T07:52:26Z","updated_at":"2016-01-19T07:52:26Z","body":"Although it looks like the new model for attributes is still expecting __toString() to work:\r\n\r\nhttps://github.com/Sylius/Sylius/blob/master/src/Sylius/Bundle/WebBundle/Resources/views/Frontend/Product/_attribute.html.twig#L1\r\n\r\nAlso this looks broken 'oldhref' so if someone is going to look at this: https://github.com/Sylius/Sylius/blob/master/src/Sylius/Bundle/AttributeBundle/Resources/views/attributeChoice.html.twig#L13\r\n\r\nI'm not sure how well tested this is with datetimes, but maybe a twig function is required to render attribute values, because casting like this won't work."}}