Class 'BBC\Autolink' not found

Topic: Class 'BBC\Autolink' not found (Read 4802 times) previous topic - next topic

0 Members and 1 Guest are viewing this topic.

Re: Class 'BBC\Autolink' not found

Reply #15 – January 01, 2017, 05:52:32 pm

Fixed.

I didn't know patch files could create commits.

Re: Class 'BBC\Autolink' not found

Reply #16 – January 01, 2017, 06:02:39 pm

Neither did I... lol

Re: Class 'BBC\Autolink' not found

Reply #17 – January 11, 2017, 12:06:07 pm

Code: [Select]

namespace BBC;

Code: [Select]

interface Database

Code: [Select]

namespace ElkArte\sources\modules;

Do not you think that it is necessary to use a single namespace for the entire project?

http://www.php-fig.org/psr/psr-4/

Example:

Code: [Select]

\Elk\Sources\Database\DB;
\Elk\Sources\Subs\BBC;
\Elk\...

(I think I already wrote about this)

Re: Class 'BBC\Autolink' not found

Reply #18 – January 11, 2017, 02:08:25 pm

https://github.com/elkarte/Elkarte/issues/2809 <= best place for this discussion!

Re: Class 'BBC\Autolink' not found

Reply #19 – January 11, 2017, 02:14:33 pm

my include path, before error

php 7.1

Code: [Select]

.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
C:\apache\php\localhost\www\elk11-fix/sources/controllers;
C:\apache\php\localhost\www\elk11-fix/sources/admin;
C:\apache\php\localhost\www\elk11-fix/addons;
C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs/CacheMethod;
\BBC;
.;
.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
C:\apache\php\localhost\www\elk11-fix/sources/controllers;
C:\apache\php\localhost\www\elk11-fix/sources/admin;
C:\apache\php\localhost\www\elk11-fix/addons;
C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs/CacheMethod;
.;
.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
C:\apache\php\localhost\www\elk11-fix/sources/controllers;
C:\apache\php\localhost\www\elk11-fix/sources/admin;
C:\apache\php\localhost\www\elk11-fix/addons;
C:\apache\php\localhost\www\elk11-fix/sources;
.;
.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
C:\apache\php\localhost\www\elk11-fix/sources/controllers;
C:\apache\php\localhost\www\elk11-fix/sources/admin;
C:\apache\php\localhost\www\elk11-fix/addons;
.;
.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
C:\apache\php\localhost\www\elk11-fix/sources/controllers;
C:\apache\php\localhost\www\elk11-fix/sources/admin;
.;
.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
C:\apache\php\localhost\www\elk11-fix/sources/controllers;
.;
.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
.;
.C:\apache\php\localhost\www\elk11-fix/sources;
.;
.;
C:\php\pear

Code: [Select]

.C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs;
C:\apache\php\localhost\www\elk11-fix/sources/controllers;
C:\apache\php\localhost\www\elk11-fix/sources/admin;
C:\apache\php\localhost\www\elk11-fix/addons;
C:\apache\php\localhost\www\elk11-fix/sources;
C:\apache\php\localhost\www\elk11-fix/sources/subs/CacheMethod;
\BBC;
C:\apache\php\localhost\www\elk11-fix/sources/subs/MentionType;

error

Re: Class 'BBC\Autolink' not found

Reply #20 – January 11, 2017, 04:04:37 pm

Did you already try the patch?

Re: Class 'BBC\Autolink' not found

Reply #21 – January 12, 2017, 04:35:50 am

how to save? there is a conflict. manually like there is no errors

Re: Class 'BBC\Autolink' not found

Reply #22 – January 12, 2017, 12:34:10 pm

Okay, back a second: are you using the beta 4 or the HEAD of the repository?

Re: Class 'BBC\Autolink' not found

Reply #23 – January 12, 2017, 01:02:58 pm

I'll wait rc1

Re: Class 'BBC\Autolink' not found

Reply #24 – January 28, 2017, 04:03:05 pm

BTW, I wrote the BBC as a root namespace because I was hoping it would become its own project.

Re: Class 'BBC\Autolink' not found

Reply #25 – January 29, 2017, 05:43:43 am

I remember, honestly the current solution is just a workaround, in 2.0 we should better fix the issue.
Also, since the idea is to have a completely separated project, the most appropriated place for the code is the ext directory.