This repository has been archived by the owner. It is now read-only.

Use the autoloader to rename form/formfield.php and form/formrule.php #633

Merged
merged 1 commit into from Dec 11, 2011

Conversation

Projects
None yet
4 participants
@chdemko
Contributor

chdemko commented Dec 11, 2011

No description provided.

@joomla-jenkins

This comment has been minimized.

Show comment
Hide comment
@joomla-jenkins

joomla-jenkins Dec 11, 2011

Build triggered by changes to the base.

Unit testing complete. There were 0 failures and 0 errors from 1883 tests and 11021 assertions.
Checkstyle analysis reported 254 warnings and 0 errors.

Build triggered by changes to the base.

Unit testing complete. There were 0 failures and 0 errors from 1883 tests and 11021 assertions.
Checkstyle analysis reported 254 warnings and 0 errors.

@realityking

This comment has been minimized.

Show comment
Hide comment
@realityking

realityking Dec 11, 2011

Contributor

Yes please! :)

Contributor

realityking commented Dec 11, 2011

Yes please! :)

@eddieajau

This comment has been minimized.

Show comment
Hide comment
@eddieajau

eddieajau Dec 11, 2011

Contributor

Does this allow us to get rid of the JFormHelper::loadFieldClass('list'); code as well?

Contributor

eddieajau commented Dec 11, 2011

Does this allow us to get rid of the JFormHelper::loadFieldClass('list'); code as well?

@realityking

This comment has been minimized.

Show comment
Hide comment
@realityking

realityking Dec 11, 2011

Contributor

I don't think so. For this we'd have to rename the fields folder to field.

Contributor

realityking commented Dec 11, 2011

I don't think so. For this we'd have to rename the fields folder to field.

@chdemko

This comment has been minimized.

Show comment
Hide comment
@chdemko

chdemko Dec 11, 2011

Contributor

no because they are stored in the 'fields' folder instead of the 'field' folder (note the "s").

Contributor

chdemko commented Dec 11, 2011

no because they are stored in the 'fields' folder instead of the 'field' folder (note the "s").

eddieajau added a commit that referenced this pull request Dec 11, 2011

Merge pull request #633 from chdemko/formautoloader
Use the autoloader to rename form/formfield.php and form/formrule.php

@eddieajau eddieajau merged commit 3cc1c62 into joomla:staging Dec 11, 2011

@eddieajau

This comment has been minimized.

Show comment
Hide comment
@eddieajau

eddieajau Dec 11, 2011

Contributor

Ok, we can attack that another time then.

Contributor

eddieajau commented Dec 11, 2011

Ok, we can attack that another time then.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.