search

rupertgermann / tt_news

TYPO3 Extension - News (tt_news)
21 stars 37 forks source link

Category tree when editing a news record no longer filters by storage Pid #58

Open LeoniePhiline opened 7 years ago

LeoniePhiline commented 7 years ago

When editing a news record, you find a category tree in the 'Categories & Relations' tab.

This category tree used to respect the storage pid - now it no longer does. In the Backend Module, however, the category tree does respect the storage pid - so different sets of categories are shown, which is very confusing for editors.

derhecht commented 7 years ago

Dear @LeoniePhiline thx for great work in #65 BUT do you think we need this feature at all? I don't like the idea to extend page table just to have this weak feature.

What do you think?

LeoniePhiline commented 7 years ago

It is a feature in tt_news right now, but it is broken. This fixes it. I think you have nothing to lose? Lots of code deal with the storagePid handling in tt_news. Your options would be to merge my pull request and everyone can move on, to keep it broken (that would seriously suck :) ) or to clean up the entire extension code and remove all the storagePid cruft.

Probably, merging is the best idea, since it has only positive effects.

LeoniePhiline commented 7 years ago

Also, the feature is actively used in a big website I maintain. ;)

derhecht commented 7 years ago

@rupertgermann could you say something about this? Whats your opinion?

marclindemann commented 7 years ago

i can confirm that this fix will work. like @LeoniePhiline say´s it´s a feature of tt_news and this should be fixxed