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

FIX: PSR-2 codebase. Formatting via phpcbf #45

Conversation

NightJar
Copy link
Contributor

@NightJar NightJar commented Dec 11, 2017

Updates the code base to be PSR-2 compliant

Closes #43

@robbieaverill
Copy link
Contributor

Needs a rebase now

@NightJar NightJar force-pushed the pulls/2.0/new-version-feed-v3 branch from 90aaa14 to 40cc9b0 Compare December 11, 2017 22:45
@robbieaverill robbieaverill changed the title Pulls/2.0/new version feed v3 FIX: PSR-2 codebase. Formatting via phpcbf Dec 11, 2017
@robbieaverill
Copy link
Contributor

Merge on green

@robbieaverill
Copy link
Contributor

Ok so we should probably remove this logic and use the ORM:

https://github.com/creative-commoners/silverstripe-versionfeed/blob/549d230adc73931a4bc52ed12e8ac17e54377998/src/VersionFeedController.php#L106-L112

But for now let's update it to _Versions or whatever the SS4 table name is so the builds will pass.

@NightJar
Copy link
Contributor Author

NightJar commented Dec 12, 2017

Yeah, there's a couple of blocks like that - the module is more heavily tied to SiteTree than I had first thought. Tests are probably passing locally because table names are case insensitive for me :/

FIX: PSR-2 codebase. Formatting via phpcbf
FIX: rendering bug in allchanges
FIX: update .gitattributes to not export codecov's config file
FIX: Update SiteTree_versions to the ss4 equivalent SiteTree_Versions
@NightJar NightJar force-pushed the pulls/2.0/new-version-feed-v3 branch from 3d80e07 to 67e112f Compare December 12, 2017 03:14
@robbieaverill
Copy link
Contributor

Builds aren't quite there yet, but as this is a PSR-2 linting pull request I think we should merge it anyway

@robbieaverill robbieaverill merged commit d536df0 into silverstripe:master Dec 12, 2017
@robbieaverill robbieaverill deleted the pulls/2.0/new-version-feed-v3 branch December 12, 2017 03:44
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

Successfully merging this pull request may close these issues.

None yet

2 participants