Issue 21 - migrate command not working

[eafarooqi]

• ->rtrim('.php') should be changed to ->replace('.php', '') as rtrim destroy the class name if p or h is the last character of class name.
• Double slash \\ included in the path, should be replaced with single slash \

Fix to this issue: #21

@DarkGhostHunter Please accept and merge, as the plugin is unusable without this fix

[coveralls]

Pull Request Test Coverage Report for Build 1634000679

• 0 of 3 (0.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 92.869%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/Migrator/Pipes/FindModelsWithSettings.php	0	3	0.0%

Totals
Change from base Build 1573764972:	0.0%
Covered Lines:	547
Relevant Lines:	589

---

💛 - Coveralls

[SalemCode8]

@DarkGhostHunter We need this fix to be able to use the package

[rudolfbruder]

@DarkGhostHunter Yeah please approve this pull request

[rudolfbruder]

@eafarooqi I tried your fix however what I am ending up with is this:

Do you think it's connected to your fix. The settable type does not have \ between the words and therefore it does not work.

However when I run $user-settings()->initialize(); it duplicates the rows of settings for that user and creates rows with proper backslashes like this:

Check the issue here

[DarkGhostHunter]

I’ll check it on new year. Italo Baeza C.

…

El 28-12-2021, a la(s) 02:53, Rudolf Bruder ***@***.***> escribió:  @eafarooqi I tried your fix however what I am ending up with is this: Do you think it's connected to your fix. The settable type does not have \ between the words and therefore it does not work. Any ideas ? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.