Added classmap for autoload

[ihortymoshenko]

No description provided.

[GromNaN]

👍

@ezyang can you merge this pull request ?

[desarrolla2]

@ezyang can you merge this pull request ? +1

[rvdbogerd]

@ezyang Can you please merge this pull request? +2

[ezyang]

Hello, can you please add a Signed-off-by, use your real email address, and get rid of the gratuitous whitespace changes?

[ihortymoshenko]

Hello, I added spaces for readability. Should I remove them?

[ezyang]

When making whitespace changes, you should separate them into a non-semantics changing whitespace change, and then a semantics change (with no whitespace changes). Sometimes it's not worth the trouble so you don't change whitespace. But diffs that touch all of the file make people sad.

I noticed you have a resubmit: HTML Purifier as a rule only accepts rebased patches on the tip of master, so you should run 'git rebase origin/master'

[ihortymoshenko]

I've done.

[ezyang]

Hmm, it doesn't look like you pushed the rebase properly. What commands did you use?

[ihortymoshenko]

git clone https://github.com/timoshenko/htmlpurifier.git
cd htmlpurifier
git remote add htmlpurifier https://github.com/ezyang/htmlpurifier.git
git fetch htmlpurifier
git rebase htmlpurifier/master
First, rewinding head to replay your work on top of it...
Applying: Added classmap for autoload
Applying: Added classmap for autoload
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
No changes -- Patch already applied.
Applying: Added classmap for autoload
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
No changes -- Patch already applied.
Applying: Added classmap for autoload
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
No changes -- Patch already applied.

[ezyang]

OK, and once you did that, what did you do to the new head? (Also, why are there so many copies of that patch on your copy?)

[ihortymoshenko]

What should I do?

[GromNaN]

@timoshenko create a new branch from @ezyang master ; add your 2 commits to this branch and create a new pull request...

[ihortymoshenko]

I removed duplicated commits using by git rebase -i HEAD~10, @ezyang can you check?

[ezyang]

I merged Rob Loach's version.

[GromNaN]

Is it required to declare the constant HTMLPURIFIER_PREFIX if the library is autoloaded with composer ?

[ezyang]

Does #12 resolve the issue for you?