This should follow same logic as #1642964: Improve file structure -- move classes to include files:

* Every class to own include files
* All views includes under /includes
* All views handles under /includes/handlers

CommentFileSizeAuthor
#9 1645188-improve-file-structure-9.patch138.91 KBamitaibu
#7 1645188-improve-file-structure-7.patch138.93 KBamitaibu
#5 1645188-improve-file-structure-5.patch137.44 KBamitaibu
#2 1645188-improve-file-structure-2.patch131.55 KBMoo64c
#1 1645188.patch0 bytesMoo64c
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Moo64c’s picture

Moo64c CreditAttribution: Moo64c commented
FileSize
1645188.patch0 bytes

(ignore this)

Moo64c’s picture

Moo64c CreditAttribution: Moo64c commented
FileSize
1645188-improve-file-structure-2.patch131.55 KB

No code change, just moving and splitting files.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Active » Needs review

Setting correct status

Status: Needs review » Needs work

The last submitted patch, 1645188-improve-file-structure-2.patch, failed testing.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Needs work » Needs review
FileSize
1645188-improve-file-structure-5.patch137.44 KB

Fixing wrong files.

Status: Needs review » Needs work

The last submitted patch, 1645188-improve-file-structure-5.patch, failed testing.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Needs work » Needs review
FileSize
1645188-improve-file-structure-7.patch138.93 KB

Patch adds includes to test files.

Status: Needs review » Needs work

The last submitted patch, 1645188-improve-file-structure-7.patch, failed testing.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Needs work » Needs review
FileSize
1645188-improve-file-structure-9.patch138.91 KB

Patch from #7 didn't load the new includes correctly.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Needs review » Fixed

Committed, thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.