Skip to content

Feature/270#22

Open
Designburo wants to merge 5 commits intoREL1_43from
feature/270
Open

Feature/270#22
Designburo wants to merge 5 commits intoREL1_43from
feature/270

Conversation

@Designburo
Copy link
Contributor

v2.7.0 rebuild-files-clean. Same as rebuild-files, but will now also physically unused files from the server

@Designburo Designburo self-assigned this Mar 10, 2026

public function __construct() {
$serverFullList = PSCore::getFilesFromServer( true );
if ( !empty( $serverFullList ) ) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Redundant, since foreach is already a no-op when the list is empty

continue;
}
$explodedFileName = explode( '_', $fileName );
$cnt = count( $explodedFileName ) - 2;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$cnt is very non-descriptive in this case

$explodedFileName = explode( '_', $fileName );
$cnt = count( $explodedFileName ) - 2;
$mergedFileName = '';
if ( $cnt >= 1 ) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is just an implode?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants