Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating article with filtered category #17974

Closed
Denitz opened this issue Sep 17, 2017 · 17 comments
Closed

Creating article with filtered category #17974

Denitz opened this issue Sep 17, 2017 · 17 comments

Comments

@Denitz
Copy link
Contributor

Denitz commented Sep 17, 2017

Steps to reproduce the issue

Set error reporting to development
Open articles manager
Filter articles by category
Create new article

Expected result

No errors

Actual result

See PHP notice:

( ! ) Notice: Array to string conversion in \administrator\components\com_categories\models\fields\categoryedit.php on line 281

System information (as much as possible)

Joomla 3.8 RC1

Additional comments

Since we have multiple categories filter, first filtered category should be pre-configured for new articles.

@ghost
Copy link

ghost commented Sep 17, 2017

Confirmed:

bildschirmfoto 2017-09-17 um 16 08 07

@alikon
Copy link
Contributor

alikon commented Sep 22, 2017

undesidered side effect from #17668

make no sense to check if "Allowed to add" on multiple categories since 1 article is related to 1 and only 1 category https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_content/controllers/article.php#L50

@infograf768
Copy link
Member

@laoneo
As the first Notice concerns the fields, could you have a look?

@laoneo
Copy link
Member

laoneo commented Sep 23, 2017

The problem is not fields itself, the issue is that JForm->getAttribute is returning an array instead of a string as it was before.

@Shazrina1994
Copy link

it always should not first filtered category should be pre-configured for new articles.
as @Denitz said under additional comments want to think of.

@Shazrina1994
Copy link

as i tested it didnt give me as selected filter order and sometimes gave me the highest categorize id when new article added.

@alikon
Copy link
Contributor

alikon commented Sep 24, 2017

test/comment #18090

@joomla-cms-bot
Copy link

Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/17974

@ghost
Copy link

ghost commented Sep 24, 2017

closed as having Pull Request #18090


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17974.

@laoneo
Copy link
Member

laoneo commented Sep 24, 2017

Does that pr fix all the issues here?

@alikon
Copy link
Contributor

alikon commented Sep 24, 2017

not sure
is a dirty one 😢

@Quy
Copy link
Contributor

Quy commented Sep 30, 2017

@franz-wohlkoenig Please reopen.

@joomla-cms-bot
Copy link

Set to "open" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/17974

@ghost
Copy link

ghost commented Sep 30, 2017

reopened as stated above.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17974.

@ggppdk
Copy link
Contributor

ggppdk commented Sep 30, 2017

I have made PR #18176

@joomla-cms-bot
Copy link

Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/17974

@ghost
Copy link

ghost commented Oct 1, 2017

closed as having Pull Request #18176


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17974.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants